[FFmpeg-soc] [soc]: r2930 - aac/aac.c

[Diego Biurrun]

On Wed, Jul 30, 2008 at 07:25:50PM +0100, Robert Swain wrote:
> 2008/7/30 Diego Biurrun <diego at biurrun.de>:
> > On Wed, Jul 30, 2008 at 05:37:41PM +0200, superdump wrote:
> >>
> >> Log:
> >> Add missing error check
> >>
> >> --- aac/aac.c (original)
> >> +++ aac/aac.c Wed Jul 30 17:37:41 2008
> >> @@ -2314,7 +2314,8 @@ static int aac_decode_frame(AVCodecConte
> >>
> >> -    output_samples(avccontext, data, data_size);
> >> +    if((err = output_samples(avccontext, data, data_size)))
> >
> > redundant ()
> 
> I'm almost certain GCC will spew a warning about the assignment if I
> don't have these parentheses, and I know how you dislike GCC warnings.
> :)

Doh, you're right of course, I overlooked this...

Diego