[prev in list] [next in list] [prev in thread] [next in thread] 

List:       rockbox-cvs
Subject:    mikmod plugin: Remove bogus doubled return statement Stick to the safer version of those two.
From:       gerrit () rockbox ! org
Date:       2014-12-22 21:36:23
Message-ID: 201412222136.sBMLaNli023201 () giant ! haxx ! se
[Download RAW message or body]

commit b681e932a9da797249ddc0e4ccab7ed7cf50fd41
Author: Thomas Jarosch <tomj@simonv.com>
Date:   Mon Dec 22 22:33:28 2014 +0100

    mikmod plugin: Remove bogus doubled return statement
    Stick to the safer version of those two.
    
    Tested mikmod still works with some nice Necros .s3m files.
    
    Change-Id: I6774cdf3345a4c89b2f9cf3bb8d832963ca30c38

diff --git a/apps/plugins/mikmod/mmio.c b/apps/plugins/mikmod/mmio.c
index 40f38d9..0ffecff 100644
--- a/apps/plugins/mikmod/mmio.c
+++ b/apps/plugins/mikmod/mmio.c
@@ -138,7 +138,6 @@ static int _mm_FileReader_Eof(MREADER* reader)
     //return feof(((MFILEREADER*)reader)->file);
 	int size   = filesize(((MFILEREADER*)reader)->file);
 	int offset = lseek(((MFILEREADER*)reader)->file, 0, SEEK_CUR);
-	return offset < 0;
 	return (size <= 0 || offset < 0 || offset >= size) ? 1 : 0;
 }
 
_______________________________________________
rockbox-cvs mailing list
rockbox-cvs@cool.haxx.se
http://cool.haxx.se/cgi-bin/mailman/listinfo/rockbox-cvs

[prev in list] [next in list] [prev in thread] [next in thread]