[MPlayer-dev-eng] [PATCH] Try twice when searching for MP3 frame header, fixes Bug 380

Guillaume POIRIER poirierg at gmail.com
Sat Dec 31 15:34:31 CET 2005


Hi,

On 12/31/05, Zuxy Meng <zuxy.meng at gmail.com> wrote:
> Hi,
>
> Currently mplayer looks for only one MP3 frame sync. The attached
> patch makes it to look for two consecutive valid MP3 frame headers,
> reducing the probability of false positives, which causes Bug 380.
>
> Funny that the fix is so simple. Seems that someone has forgotten to
> initialize MP3_resync correctly.

What do you think about this patch? It's trivial, and apart from maybe
slowing down a bit initialisation, killing one more bugzilla is sweet!
:P

Guillaume
--
Life is pleasant. Death is peaceful. It's the transition that's troublesome.
    Isaac Asimov (1920 - 1992)

MPlayer's doc is offline. Find some fresh one here:
http://tuxrip.free.fr//MPlayer-DOCS-HTML/en/
http://tuxrip.free.fr//MPlayer-DOCS-HTML/fr/




More information about the MPlayer-dev-eng mailing list