doxygen.sh: Exclude private header by default
Sebastian Kügler
sebas at kde.org
Fri May 28 14:31:33 BST 2010
On Thu May 27 2010 18:43:18 John Layt wrote:
> On Wednesday 26 May 2010 20:32:04 Allen Winter wrote:
> > On Wednesday 26 May 2010 2:59:14 pm Sebastian Trüg wrote:
> > > May I commit the attached patch which excludes all private (*_p.h)
> > > headers by default? This, for example cleans up the Plasma docs.
> >
> > Actually, I don't remember the context now.. I think John Layt reported
> > problems to me with having the _p.h files excluded by default.
>
> To quote my e-mail:
>
> "I've noticed the apidox BADDOC and MATCHCLASS checks appear to have a
> problem whenever the methods have been declared in a header file whose
> name doesn't exactly match that of the .ccp file, e.g. if it's in a _p.h
> private header file instead. This is throwing up a lot of false
> positives, probably over a 1000 in kdelibs alone. I realise it's a
> doxygen problem, but is there anything we can do to filter these out?"
>
> So it's a case of which produces the most false positives, I think we get
> fewer with them included. Besides, why shouldn't private api also come up
> to scratch, will you really remember what every internal class does in 5
> years time? :-)
People using those APIDOX are mostly those who'll never use the private API,
so I'd say to accommodate for the primary usecases, private classes only add
clutter and confusion.
--
sebas
http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
More information about the kde-core-devel
mailing list