On Tue, Jul 15, 2008 at 04:03:03PM +0100, Robert Swain wrote:
> 2008/7/15 Benjamin Larsson <[EMAIL PROTECTED]>:
> > superdump wrote:
> >> Author: superdump
> >> Date: Tue Jul 15 15:47:11 2008
> >> New Revision: 2791
> >>
> >> Log:
> >> Add some allocation checks
> >>
> >
> > There should be a NOMEM error. Look in cook.c for reference.
> 
> Or rather atrac3.c or so. How about the attached? I had to add code to
> propagate the error values through to the main functions as I assume
> only the return values of the init, decode, encode and close functions
> really matter.
[...]

> -        if(program_config_element_default(ac, channels) < 0)
> +        if((ret = program_config_element_default(ac, channels)))
>              return -1;

shouldnt that be return ret?
Also the <0 disappeared, was that intended?

[...]


-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates

Attachment: signature.asc
Description: Digital signature

_______________________________________________
FFmpeg-soc mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc

Reply via email to