Quoting wm4 (2015-04-11 13:19:45) > On Sat, 14 Mar 2015 21:41:13 +0530 > Himangi Saraogi <[email protected]> wrote: > > > --- > > libavformat/ac3dec.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavformat/ac3dec.c b/libavformat/ac3dec.c > > index 4ceffa5..38a2a8f 100644 > > --- a/libavformat/ac3dec.c > > +++ b/libavformat/ac3dec.c > > @@ -61,7 +61,7 @@ static int ac3_eac3_probe(AVProbeData *p, enum AVCodecID > > expected_codec_id) > > > > if (max_frames) { > > int pes = 0, i; > > - unsigned int code = -1; > > + unsigned int code = 0xffffffff; > > > > #define VIDEO_ID 0x000001e0 > > #define AUDIO_ID 0x000001c0 > > Why change this? AFAIK it's perfectly valid C.
It's valid, but I think it can be rather confusing if you're not familiar with this idiom. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
