On Fri, 02 Nov 2012 01:06:33 +0100, Diego Biurrun <di...@biurrun.de> wrote: > On Wed, Oct 31, 2012 at 09:26:10PM +0100, Anton Khirnov wrote: > > On Wed, 31 Oct 2012 17:33:35 +0100, Diego Biurrun <di...@biurrun.de> wrote: > > > On Mon, Oct 29, 2012 at 03:45:43PM +0100, Diego Biurrun wrote: > > > > On Mon, Oct 29, 2012 at 03:19:27PM +0100, Janne Grunau wrote: > > > > > On 2012-10-29 15:03:45 +0100, Diego Biurrun wrote: > > > > > > On Sun, Oct 28, 2012 at 01:34:55AM +0200, Janne Grunau wrote: > > > > > > > Documentation includes only the externally visible API of the > > > > > > > installed > > > > > > > headers. > > > > > > > > > > > > What about the rest? > > > > > > > > > > doxygen doc/Doxyfile > > > > > > > > > > If you prefer make support for propose a output directory and a > > > > > target > > > > > name. > > > > > > > > We could have > > > > > > > > doc/doxy/api/ > > > > doc/doxy/all/ > > > > > > > > or similar. > > > > > > > > Note that this patch only extracts documentation from .h files. I > > > > suspect that some part of the documentation for the stuff declared in > > > > those headers is in .c files. > > > > > > Here's a list of files which are unconditionally compiled and > > > contain more than just @file Doxygen: > > > > > > libavcodec/parser.c > > > > Only for av_parser_change, which should be removed anyway. > > Removing a function requires a major bump, so it will take some time. > > > > libavcodec/resample.c > > > libavcodec/resample2.c > > > > deprecated > > So? > > > I.e. all except crc.c false positives. > > No, the resample and parser stuff are not false positives. >
The point is, those functions should not be used, so there is little point in documenting them. Feel free to move the doxy of course, but i consider it a waste of time. -- Anton Khirnov _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel
