[FFmpeg-devel] [PATCH] Codec lookup: do not use codec_id

Michael Niedermayer michaelni
Mon Aug 6 16:59:15 CEST 2007


Hi

On Mon, Aug 06, 2007 at 03:05:26PM +0200, Benoit Fouet wrote:
> Hi Nicolas,
> 
> Nicolas George wrote:
> > Le nonidi 19 thermidor, an CCXV, Benoit Fouet a ?crit :
> >   
> >> i don't see why this should be addressed separately.
> >>     
> >
> > I guess my flair for what must be in a separate patch needs to be sharpened
> > some more.
> >
> > Here is the updated version of the patch. Compared to yesterday version, it
> > only adds seven av_free; it passes the same tests.
> >
> >   
> 
> i personnaly prefer this one...
> Michael ?

instead of 
av_free(x)
x=NULL

av_freep(&x)
could be used

besides that iam fine with either patch

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

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070806/acddeb65/attachment.pgp>



More information about the ffmpeg-devel mailing list