[PATCH] D34091: Support for querying the exception specification type through libclang

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 15 14:27:14 PDT 2017


aaron.ballman added inline comments.


================
Comment at: include/clang-c/Index.h:213
+  /**
+   * \brief The exception specification has not yet been evaluated
+   */
----------------
This comment is now missing the full-stop at the end of the sentence (you dropped one too many periods).


================
Comment at: tools/libclang/CXType.cpp:693
+
+    if (const FunctionProtoType* FD = T->getAs<FunctionProtoType>()) {
+        return static_cast<int>(FD->getExceptionSpecType());
----------------
aaron.ballman wrote:
> Use `const auto *` and elide the braces.
You should run your patch through clang-format; the asterisk should bind to `FD` rather than `auto` and the indentation is too large in the patch.


https://reviews.llvm.org/D34091





More information about the cfe-commits mailing list