Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: WPS playlist viewer exposes a bug in either plaback/buffering or playlist. I'm stumped!

Re: WPS playlist viewer exposes a bug in either plaback/buffering or playlist. I'm stumped!

From: Jonathan Gordon <jdgordy_at_gmail.com>
Date: Thu, 22 Apr 2010 17:19:33 +1000

On 16 April 2010 17:46, Marcin Bukat <marcin.bukat_at_gmail.com> wrote:
> 2010/4/16 Jonathan Gordon <jdgordy_at_gmail.com>:
>> Hi all,
>> Well I'm stumped on this bug
>> (http://www.rockbox.org/tracker/task/11175 ). Basically what happens
>> is if you use a wps with the playlist viewer (attached should work on
>> every target) the playlist will load in the wrong order and Wierd Shit
>> (TM) happens when jumping around tracks. If you use a theme without it
>> (or even I assume the same theme with it disabled (i.e in a
>> conditional viewport)) the playlist would work fine.
>>
>> The code for the playlist viewer is draw_playlist_viewer_list() in
>> apps/gui/skin_engine/skin_display.c starting around line 182. I'm
>> pretty certain the problem is either in audio_peek_track() being
>> called or playlist_peek(). Nothing else there touchees anything
>> outside of the drawing/skln code.
>>
>> I dont know playlist/buffering/playback anywhere near enough to figure
>> out what's going on, so if someone who does could have a look, that
>> would be great.
>>
>> We havnt managed to repro in the sim which makes things a bit harder.
>>
>> Cheers and have a good weekend.
>>
>> Jonathan
>>
> If this wps will work on my MPIO HD200 I can look at the issue with
> BDM/gdb combo but I don't know if it is good to debug this on
> unfinished port which has its own problems.
>
> wo
>

I apparently missed this reply.
Attached is a .wps which will work on any target.
have repeat and shuffle off, then start a dirplay and make sure the
tracks are displayed in order. if they arnt then you have repro.

  • application/vnd.ms-works attachment: plv.wps
Received on 2010-04-22

Page was last modified "Jan 10 2012" The Rockbox Crew
aaa