diff options
author | albert-github <albert.tests@gmail.com> | 2019-03-30 09:43:15 (GMT) |
---|---|---|
committer | albert-github <albert.tests@gmail.com> | 2019-03-30 09:43:15 (GMT) |
commit | fe8a55092e6f890792f6ff8aeecfc1dce86160a0 (patch) | |
tree | 60e3bd77efd5284e2e6b9c954bcd8ce2bb7ca08e /src | |
parent | 8f04d2e35f6a4c77452603852838efd007c4fa24 (diff) | |
download | Doxygen-fe8a55092e6f890792f6ff8aeecfc1dce86160a0.zip Doxygen-fe8a55092e6f890792f6ff8aeecfc1dce86160a0.tar.gz Doxygen-fe8a55092e6f890792f6ff8aeecfc1dce86160a0.tar.bz2 |
Bug 685714 - false positives reporting parameters or return value not being documented
The "validating" routines should always be called.
They were missed out in case of:
- only XML format there should be warnings as well (XML has its own write routines)
- no formats there should be warnings as well
Diffstat (limited to 'src')
-rw-r--r-- | src/outputlist.cpp | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/src/outputlist.cpp b/src/outputlist.cpp index 93a1b6e..6e6d9ef 100644 --- a/src/outputlist.cpp +++ b/src/outputlist.cpp @@ -143,12 +143,16 @@ bool OutputList::generateDoc(const char *fileName,int startLine, { if (og->isEnabled()) count++; } - if (count==0) return TRUE; // no output formats enabled. + // we want to validate irrespective of the number of output formats + // specified as: + // - when only XML format there should be warnings as well (XML has its own write routines) + // - no formats there should be warnings as well DocRoot *root=0; root = validatingParseDoc(fileName,startLine, ctx,md,docStr,indexWords,isExample,exampleName, singleLine,linkFromIndex); + if (count==0) return TRUE; // no output formats enabled. writeDoc(root,ctx,md); @@ -181,10 +185,15 @@ bool OutputList::parseText(const QCString &textStr) { if (og->isEnabled()) count++; } - if (count==0) return TRUE; // no output formats enabled. + // we want to validate irrespective of the number of output formats + // specified as: + // - when only XML format there should be warnings as well (XML has its own write routines) + // - no formats there should be warnings as well DocText *root = validatingParseText(textStr); + if (count==0) return TRUE; // no output formats enabled. + for (it.toFirst();(og=it.current());++it) { if (og->isEnabled()) og->writeDoc(root,0,0); |