[PATCH] Last-chance diagnostics for incomplete types
James Dennett
jdennett at google.com
Fri Jul 26 16:52:59 PDT 2013
================
Comment at: include/clang/Sema/MultiplexExternalSemaSource.h:325
@@ -324,1 +324,3 @@
+ virtual bool MaybeDiagnoseMissingCompleteType(SourceLocation Loc, QualType T);
+
----------------
Please document this, even if it's just inheriting docs and then saying how this implements the interface.
================
Comment at: include/clang/Sema/ExternalSemaSource.h:183
@@ +182,3 @@
+ /// \brief Produces a diagnostic note if the external source contains a
+ /// complete definition for T.
+ ///
----------------
Nit: "\p T" -> "T" to refer to the parameter T.
http://llvm-reviews.chandlerc.com/D1213
More information about the cfe-commits
mailing list