[MPlayer-dev-eng] [PATCH] Support layers 1 and 2 in mp3 demuxer

D Richard Felker III dalias at aerifal.cx
Sat Apr 2 18:30:08 CEST 2005


On Sat, Apr 02, 2005 at 01:32:56PM +0200, Nico Sabbi wrote:
> Nico Sabbi wrote:
> 
> >
> >committed a much smaller and more correct patch.
> >Yet, in demux_audio I should set sh_audio->format=0x50 in case of 
> >layers 1 and 2.
> 
> 
> This patch assigns sh_audio->audio.dwScale, sh_audio->wf->nBlockAlign 
> and sh_audio->format
> according to the layer and the sampling frequency; it also modifies the 
> seeking functions accordingly.
> 
> Rich, is it ok to commit?

It looks ok at a glance.
When patching demux_audio, could you make sure to always check
-hr-mp3-seek and make sure you haven't broken it. This is the most
common error people seem to make when modifying that file..

Rich




More information about the MPlayer-dev-eng mailing list