kurioes / pmix

Automatically exported from code.google.com/p/pmix
0 stars 0 forks source link

PMix exception flood when use it with MPD after 0.15 #8

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
(in any order)
1. Start PMix and connect to mpd.
2. Start playing.

What is the expected output? What do you see instead?
No exceptions in adb logcat. I see a lot of
org.a0z.mpd.InvalidResponseException exceptions.

What version of the product are you using? On what operating system?
PMix 0.3 and PMix 0.4-beta.
mpd 0.15~beta2.

Please provide any additional information below.
# adb logcat
....
W/System.err( 3162): org.a0z.mpd.InvalidResponseException: unknown
response: single: 0
W/System.err( 3162):    at org.a0z.mpd.MPDStatus.<init>(MPDStatus.java:152)
W/System.err( 3162):    at org.a0z.mpd.MPD.getStatus(MPD.java:421)
W/System.err( 3162):    at
org.a0z.mpd.MPDStatusMonitor.run(MPDStatusMonitor.java:85)
W/System.err( 3162): org.a0z.mpd.InvalidResponseException: unknown
response: consume: 0
W/System.err( 3162):    at org.a0z.mpd.MPDStatus.<init>(MPDStatus.java:152)
W/System.err( 3162):    at org.a0z.mpd.MPD.getStatus(MPD.java:421)
W/System.err( 3162):    at
org.a0z.mpd.MPDStatusMonitor.run(MPDStatusMonitor.java:85)
W/System.err( 3162): org.a0z.mpd.InvalidResponseException: unknown
response: nextsong: 6
W/System.err( 3162):    at org.a0z.mpd.MPDStatus.<init>(MPDStatus.java:152)
W/System.err( 3162):    at org.a0z.mpd.MPD.getStatus(MPD.java:421)
W/System.err( 3162):    at
org.a0z.mpd.MPDStatusMonitor.run(MPDStatusMonitor.java:85)
W/System.err( 3162): org.a0z.mpd.InvalidResponseException: unknown
response: nextsongid: 6
W/System.err( 3162):    at org.a0z.mpd.MPDStatus.<init>(MPDStatus.java:152)
W/System.err( 3162):    at org.a0z.mpd.MPD.getStatus(MPD.java:421)
W/System.err( 3162):    at
org.a0z.mpd.MPDStatusMonitor.run(MPDStatusMonitor.java:85)
W/System.err( 3162): org.a0z.mpd.InvalidResponseException: unknown
response: single: 0
W/System.err( 3162):    at org.a0z.mpd.MPDStatus.<init>(MPDStatus.java:152)
W/System.err( 3162):    at org.a0z.mpd.MPD.getStatus(MPD.java:421)
W/System.err( 3162):    at
org.a0z.mpd.MPDStatusMonitor.run(MPDStatusMonitor.java:85)
W/System.err( 3162): org.a0z.mpd.InvalidResponseException: unknown
response: consume: 0
W/System.err( 3162):    at org.a0z.mpd.MPDStatus.<init>(MPDStatus.java:152)
W/System.err( 3162):    at org.a0z.mpd.MPD.getStatus(MPD.java:421)
W/System.err( 3162):    at
org.a0z.mpd.MPDStatusMonitor.run(MPDStatusMonitor.java:85)
W/System.err( 3162): org.a0z.mpd.InvalidResponseException: unknown
response: nextsong: 6
W/System.err( 3162):    at org.a0z.mpd.MPDStatus.<init>(MPDStatus.java:152)
W/System.err( 3162):    at org.a0z.mpd.MPD.getStatus(MPD.java:421)
W/System.err( 3162):    at
org.a0z.mpd.MPDStatusMonitor.run(MPDStatusMonitor.java:85)
W/System.err( 3162): org.a0z.mpd.InvalidResponseException: unknown
response: nextsongid: 6
W/System.err( 3162):    at org.a0z.mpd.MPDStatus.<init>(MPDStatus.java:152)
W/System.err( 3162):    at org.a0z.mpd.MPD.getStatus(MPD.java:421)
W/System.err( 3162):    at
org.a0z.mpd.MPDStatusMonitor.run(MPDStatusMonitor.java:85)

....

So we can see that mpd has some new responses on 'status' command. I have
added a patch that adds those to MPDStatus, but it only fixes this issue
and doesn't add any new functionality (single play, next song display and
so on).

Original issue reported on code.google.com by andrew.a.smirnoff on 20 Jul 2009 at 5:45

Attachments:

GoogleCodeExporter commented 9 years ago
I second this issue - am currently running own build of pmix on my G1 with
essentially the same changes as detailed in the patch above.

In order to correctly deal with these responses from MPD the settings are as 
follows:

  * 'single' mode plays song only once.
  * 'repeat' mode repeats each song (!)
  * 'consume' mode deletes each played song.

http://www.mail-archive.com/musicpd-dev-team@lists.sourceforge.net/msg00774.html

Original comment by mafro...@gmail.com on 23 Jul 2009 at 7:41

GoogleCodeExporter commented 9 years ago
Should be fixed with 0.4-beta2 (thanks for the patch)

Original comment by stefan.a...@gmail.com on 8 Sep 2009 at 11:21