[FFmpeg-devel] [PATCH] Indeo5 decoder
Sat Mar 28 13:50:50 CET 2009
On Fri, Mar 27, 2009 at 04:45:15PM +0000, M?ns Rullg?rd wrote:
> Diego Biurrun <diego at biurrun.de> writes:
> > On Fri, Mar 27, 2009 at 04:41:18PM +0100, Maxim wrote:
> >> Any further reviews plz!
> > Your patch is missing a documentation and changelog update.
> >> --- libavcodec/indeo5.c (Revision 0)
> >> +++ libavcodec/indeo5.c (Revision 0)
> >> @@ -0,0 +1,846 @@
> >> +
> >> +/**
> >> + * @file indeo5.c
> > Add the subdirectory prefix, i.e. libavcodec/.
> What are these tags good for anyway?
doxygen uses them to generate documentation in html or tex and possibly
a few other formats
google pointed to http://drv.nu/ffmpeg/doxygen/ for ffmpeg and doxygen
if you want to see the result.
Also it has to be kept in mind that noone checks the doxy output when
changing comments and the doxy configuration file has not been maintained
for years, so this surely could look better
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
I wish the Xiph folks would stop pretending they've got something they
do not. Somehow I fear this will remain a wish. -- M?ns Rullg?rd
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel