[Ffmpeg-devel] [PATCH] libavcodec/utils.c Doxygen docs

[Michael Niedermayer]

Hi

On Tue, Feb 20, 2007 at 12:22:04PM +0100, Panagiotis Issaris wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Hi,
> 
> The attached patch adds some more Doxygen documentation to
> libavcodec/utils.c.
[...]
> @@ -765,8 +770,11 @@
>  }
>  
>  /**
> - * allocates a AVCodecContext and set it to defaults.
> - * this can be deallocated by simply calling free()
> + * Allocates an AVCodecContext and sets its fields to default values.  The
> + * resulting struct can be deallocated by simply calling free().
[...]
> + * Allocates an AVFrame and sets its fields to default values.  The resulting
> + * struct can be deallocated by simply calling free().

av_free()


[...]
> +/**
> + * Attempts to find an encoder with a matching codec ID.

id vote for removing of the attempt, you could add it in front of anything
which can fail ...

iam fine with it otherwise, no need to post anoter patch just commit

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

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070220/d297d038/attachment.pgp>