[FFmpeg-devel] [PATCH] swscale-example: Use av_{free, malloc} instead of the system ones.

Michael Niedermayer michaelni
Sun Aug 23 23:24:24 CEST 2009


On Sun, Aug 23, 2009 at 06:19:43PM -0300, Ramiro Polla wrote:
> On Sun, Aug 23, 2009 at 6:10 PM, Michael Niedermayer<michaelni at gmx.at> wrote:
> > On Sun, Aug 23, 2009 at 05:52:36PM -0300, Ramiro Polla wrote:
> >> On Sun, Aug 23, 2009 at 12:27 AM, Michael Niedermayer<michaelni at gmx.at> wrote:
> >> > On Sat, Aug 22, 2009 at 05:30:14PM -0300, Ramiro Polla wrote:
> >> >> $subj
> >> >
> >> >> ?swscale-example.c | ? 16 ++++++++--------
> >> >> ?1 file changed, 8 insertions(+), 8 deletions(-)
> >> >> 70f7e63ec4ab57403f986ccb76bbbb5239b18887 ?0005-Use-av_-free-alloc-instead-of-the-system-ones.patch
> >> >> From 02fd6052b28190df853c04ba8ed19f848e9808a2 Mon Sep 17 00:00:00 2001
> >> >> From: Ramiro Polla <ramiro.polla at gmail.com>
> >> >> Date: Sat, 22 Aug 2009 16:24:51 -0300
> >> >> Subject: [PATCH] Use av_{free,alloc} instead of the system ones.
> >> >
> >> > whats the point?
> >>
> >> Remove an if (data) to free data =)
> >
> > why would you need if(data) ?
> 
> Aren't there some libc's that don't check for NULL inside free()?

we will see

seriously, example code should be simple and useage of av_free* may not be
for someone unfamiliar with libav* development

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

Let us carefully observe those good qualities wherein our enemies excel us
and endeavor to excel them, by avoiding what is faulty, and imitating what
is excellent in them. -- Plutarch
-------------- 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/20090823/d4d0e10e/attachment.pgp>



More information about the ffmpeg-devel mailing list