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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to