[FFmpeg-devel] [PATCH] fix small memleak in rdt.c
Sat Nov 15 00:49:06 CET 2008
On Fri, Nov 14, 2008 at 03:04:20PM -0800, Baptiste Coudurier wrote:
> Michael Niedermayer wrote:
> > On Fri, Nov 14, 2008 at 02:04:00PM -0800, Baptiste Coudurier wrote:
> >> Michael Niedermayer wrote:
> >>> On Fri, Nov 14, 2008 at 03:17:26PM -0500, Ronald S. Bultje wrote:
> >>>> Hi,
> >>>> $subj.
> >>>> Sidenote: if url_open_buf() allocates, shouldn't url_close_buf() free
> >>>> it? That doesn't appear to be the case right now, hence my confusion
> >>>> when I originally implemented this... Maybe the API should be renamed
> >>>> or something? Or a comment in the API docs?
> >>> id say mark both as deprecated and put them under #if VERSION < X
> >>> close is unused, open is just used by ffserver.c
> >> Humm :/
> >> Well is it possible then to rename init_put_byte, etc... to more decent
> >> names ?
> >> Like: av_alloc_byteio_context, av_init_byteio_context ?
> > isnt the existing av_alloc_put_byte() enough?
> Its name is inappropriate and inconsistent with other lavf functions,
> this structure is also used to read for example.
hmm, i dont mind if its changed with the next major bump ...
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel