[FFmpeg-devel] [PATCH 1/4] lavu: add simple array implementation

Michael Niedermayer michaelni at gmx.at
Mon Mar 3 10:39:42 CET 2014

On Mon, Mar 03, 2014 at 01:59:43AM +0100, Lukasz Marek wrote:
> >You're adding it to lavu, and it's not marked as internal/experimental. If
> >it stinks or needs 3 API updates or re-implementations, the old cruft
> >sticks around for years to come.
> Can you precise how to mark it? I added it as public, but don't
> understand how can i make it experimental.

> >That's why we now have 2 (and with yours, soon 3) array implementations.
> >Can we stick to one? At least java uses the same API for all its array/list
> >things.
> The existing two doesn't allow to free "objects" with allocated
> pointers inside. If you fix them then you have 4. I really think it
> is better to have one common for wide range of use cases. I can
> refactor existing code to use mine if accepted.

if i have an array of "object" with pointers inside them and want
to free them id do

for(i=0; i<arraysize; i++)
    my_free(my_array + i);

that needs no API beyond ANSI/ISO C
what am i missing ?

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The bravest are surely those who have the clearest vision
of what is before them, glory and danger alike, and yet
notwithstanding go out to meet it. -- Thucydides
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140303/0d40608c/attachment.asc>

More information about the ffmpeg-devel mailing list