Closed GoogleCodeExporter closed 8 years ago
Hmm I see what this patch is doing but I am wondering if it is because of an
invalid way forke-daapd is sending the data and not so much with the parser?
What I am saying is the bug possibly in forked-daapd and not in TunesRemote+?
Original comment by mellowaredev
on 19 Jul 2011 at 3:10
Hi, That could well be the case but the small bit of extra exception handling
in the parser does seem to fix things and I don't think it would introduce any
issues for other servers :)
I was also having problems with the pause functionality with TunesRemote+ with
forked-daapd. Separating out the pause / playpause calls seems to help. I've
managed to patch that also if you would like to see that patch, I'm not sure
how it would impact other servers though as I've nothing extra to test with.
The final issue that I have is regarding the search, I am still looking into
that to see what to problem is.
Original comment by david%co...@gtempaccount.com
on 19 Jul 2011 at 3:40
No I appreciate the work. Attach any patches you want and I can always test
them against MonkeyTunes or iTunes to make sure nothing else is broken.
Original comment by mellowaredev
on 19 Jul 2011 at 3:49
Thanks that's much appreciated,
Here is a patch that contains both the artist list and play / pause fixes.
I'll attach another when I get to the bottom of the search failure.
Original comment by david%co...@gtempaccount.com
on 19 Jul 2011 at 4:02
Attachments:
I have incorporated your patches and so far so good but i have created a BETA
APK and put it on the downloads page for others to try to make sure nothing is
broken...
Original comment by mellowaredev
on 20 Jul 2011 at 12:14
Will give it a shot this weekend, and let you know.
Original comment by cramermaurice@gmail.com
on 22 Jul 2011 at 2:36
Tried it out for a while now, and didn't experience any problems with the
latest Beta.
Original comment by cramermaurice@gmail.com
on 2 Aug 2011 at 1:07
Released on Android Market v 2.4.1
Original comment by mellowaredev
on 24 Aug 2011 at 8:42
Original issue reported on code.google.com by
david%co...@gtempaccount.com
on 19 Jul 2011 at 1:34Attachments: