[PATCH] D47157: Warning for framework headers using double quote includes

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 31 12:40:35 PDT 2018


dexonsmith added inline comments.


================
Comment at: test/Modules/double-quotes.m:27-29
+// CHECK: double-quoted include "A0.h" in framework header, expected angle-bracketed include <A/A0.h> instead
+// CHECK: double-quoted include "B.h" in framework header, expected angle-bracketed include <B.h> instead
+// CHECK: double-quoted include "B.h" in framework header, expected angle-bracketed include <B.h> instead
----------------
aaron.ballman wrote:
> dexonsmith wrote:
> > When there's a fixit, you don't need to list it in the warning text (the fix-it itself is sufficient).  I also feel like "quoted include" is as clear as "double-quoted include" (but more succinct).  So I think these would be better as:
> > 
> > > warning: quoted include "..." in framework header, expected angle-bracketed include instead
> > 
> Some other lexer diagnostics use "double-quoted" when they want to distinguish with "angle-bracketed" (see `warn_pragma_include_alias_mismatch_angle` and `warn_pragma_include_alias_mismatch_quote` as examples). I don't have a strong opinion on what form we use, but I'd prefer for it to be consistent exposition.
I agree we should be consistent.  No reason to change it here then.


https://reviews.llvm.org/D47157





More information about the cfe-commits mailing list