the simulator code now deals better with playlist_next() returning NULL

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@1941 a1c6a512-1295-4272-9138-f99709370657

1 files changed, 13 insertions, 9 deletions

diff --git a/firmware/mpeg.c b/firmware/mpeg.c
index e7c3638da9..6f2b3dfafe 100644
--- a/firmware/mpeg.c
+++ b/firmware/mpeg.c
@@ -1246,17 +1246,17 @@ void mpeg_play(int offset)
 
     do {
         trackname = playlist_next( steps, NULL );
-        if ( trackname ) {
-            if(mp3info(&taginfo, trackname)) {
-                /* bad mp3, move on */
-                steps++;
-                continue;
-            }
-            taginfo.offset = offset;
-            set_elapsed(&taginfo);
-            playing = true;
+        if (!trackname)
             break;
+        if(mp3info(&taginfo, trackname)) {
+            /* bad mp3, move on */
+            steps++;
+            continue;
         }
+        taginfo.offset = offset;
+        set_elapsed(&taginfo);
+        playing = true;
+        break;
     } while(1);
 #else
     queue_post(&mpeg_queue, MPEG_PLAY, (void*)offset);
@@ -1302,6 +1302,8 @@ void mpeg_next(void)
 
     do {
         file = playlist_next(steps, NULL);
+        if(!file)
+            break;
         if(mp3info(&taginfo, file)) {
             steps++;
             continue;
@@ -1323,6 +1325,8 @@ void mpeg_prev(void)
 
     do {
         file = playlist_next(steps, NULL);
+        if(!file)
+            break;
         if(mp3info(&taginfo, file)) {
             steps--;
             continue;