[FFmpeg-devel] [PATCH 5/7] mp3dec: mark as AVSTREAM_PARSE_FULL_RAW

Michael Niedermayer michaelni at gmx.at
Thu Jul 26 02:59:11 CEST 2012


On Tue, Jul 24, 2012 at 09:35:42PM +0200, Reimar Döffinger wrote:
> On Tue, Jul 24, 2012 at 06:15:37PM +0200, Michael Niedermayer wrote:
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > ---
> >  libavformat/mp3dec.c |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/libavformat/mp3dec.c b/libavformat/mp3dec.c
> > index 5300ab1..f6db77a 100644
> > --- a/libavformat/mp3dec.c
> > +++ b/libavformat/mp3dec.c
> > @@ -167,7 +167,7 @@ static int mp3_read_header(AVFormatContext *s)
> >  
> >      st->codec->codec_type = AVMEDIA_TYPE_AUDIO;
> >      st->codec->codec_id = CODEC_ID_MP3;
> > -    st->need_parsing = AVSTREAM_PARSE_FULL;
> > +    st->need_parsing = AVSTREAM_PARSE_FULL_RAW;
> 
> Shouldn't for consistency all formats be switched to use that?
> In particular in rawdec (ff_raw_audio_read_header) and wav?

switched


> Additional possible candidates: aacdec, loasdec, omadec.

ill try to switch them after this patchset, dont want to make too
many changes in the middle of a tested patchset ...

patch applied

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120726/f34253ff/attachment.asc>


More information about the ffmpeg-devel mailing list