[FFmpeg-devel] [PATCH] Indeo5 decoder

Michael Niedermayer michaelni
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
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090328/3fb57146/attachment.pgp>



More information about the ffmpeg-devel mailing list