[MPlayer-cvslog] CVS: main/libmpdemux mp3_hdr.c,1.8,1.9

Rich Felker dalias at aerifal.cx
Sun Apr 17 17:59:44 CEST 2005


On Sat, Apr 02, 2005 at 12:30:21PM +0200, Nico Sabbi CVS wrote:
> CVS change done by Nico Sabbi CVS
> 
> Update of /cvsroot/mplayer/main/libmpdemux
> In directory mail:/var2/tmp/cvs-serv29242
> 
> Modified Files:
> 	mp3_hdr.c 
> Log Message:
> added support for mpa layers 1 and 2

This patch is broken, or else reveals another bug that was covered up
before. I have a 128kbit cbr mp3 file which, when played with this
patch, fails to read entirely. A verbose debug run shows mplayer
misdetecting its frames as alternating wildly between layers 1 and 3,
but mp3_check reports no errors in the file. A short clip is available
at:

http://brightrain.aerifal.cx/~dalias/mp3_hdr.c-1.9-sux.mp3

Rich





More information about the MPlayer-cvslog mailing list