[FFmpeg-devel] [PATCH] Document sws_freeContext()
Mon Nov 2 01:39:32 CET 2009
On Mon, Nov 02, 2009 at 12:30:16AM +0100, Stefano Sabatini wrote:
> Hi, as in subject.
> FFmpeg = Fantastic & Fast Mysterious Philosophical Elitarian Governor
> swscale.h | 4 ++++
> 1 file changed, 4 insertions(+)
> 76c99c975c1aa2f36453e7d11a6e2c8d8f6f8dc2 document-sws-free-context.patch
> Index: libswscale/swscale.h
> --- libswscale/swscale.h (revision 29810)
> +++ libswscale/swscale.h (working copy)
> @@ -112,6 +112,10 @@
> struct SwsContext;
> + * Frees the memory allocated by swsContext.
> + * If swsContext is NULL, then do nothing.
sounds odd, id say either
free + do
frees + does
(just gut feeling)
Also it does not just free memory and not what has been
allocated by swsContext, as thats a struct and not a function
that does something.
really i think that
"frees a swscaler context" is perfectly fine
if one wants to write more text one needs to write this more general
to stay correct. Like freeing all resources (not just memory) that
have been allocated during context creation as well as possibly later.
but this appears to be heading in the overkill directtion IMHO
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel