[PATCH] D64696: Adds a warning when an inline Doxygen comment has no argument

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 19 07:48:23 PDT 2019


gribozavr added a comment.

Is it true that all inline commands require an argument?

For example, "\&" does not.



================
Comment at: clang/test/Sema/warn-documentation.cpp:1030
 
+// The inline comments expect a string after the command.
+// expected-warning at +1 {{'\a' command does not have an argument}}
----------------
Please move this new section below the test_attach tests.


================
Comment at: clang/test/Sema/warn-documentation.cpp:1033
+/// \a
+int test_inline_no_argument_a_b(int);
+
----------------
Please un-abbreviate b and g into bad and good.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64696/new/

https://reviews.llvm.org/D64696





More information about the cfe-commits mailing list