diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2017-01-19 18:01:57 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2017-01-19 18:03:26 +0100 |
commit | 28ec4d1456ae17711749e4131e9f8e96eccb1b95 (patch) | |
tree | 7064ddc3136e999de5623d4cf25c0936b01a76c3 /cpputools | |
parent | 416252b1f6d60d36f8cb0b57c71a0edf33395a44 (diff) |
New loplugin:dynexcspec: Add @throws documentation
See the mail thread starting at
<https://lists.freedesktop.org/archives/libreoffice/2017-January/076665.html>
"Dynamic Exception Specifications" for details.
* The check for missing @throws documentation is not too specific, it just
checks whether a function with dynamic exception specification has /any/
@throws clause, not necessarily exactly matching the exception types. (Many
of the details in the existing dynamic exception specifications are probably
not very useful, anyway.)
* When adding @throws clauses, I bluntly copied the exception specifications
except for dropping any mentions of std::exception (except in the rare cases
where that was the only exception typed mentioned).
* In many places it might have looked more natural to use trailing Doxygen
comments of the
///< @throws ...
kind, but Clang's getCommentForDecl unfortunately doesn't detect trailing
comments on function decls.
* Also, Clang's getCommentForDecl doesn't look into macros, so some trivial
silly macros were expanded along the way to add comments where necessary.
Change-Id: I1831d72df2d9c801d4b8dd7d708d9cefea039589
Diffstat (limited to 'cpputools')
0 files changed, 0 insertions, 0 deletions