MyDeveloperDay added inline comments.

================
Comment at: clang/unittests/Format/ObjCPropertyAttributeOrderFixerTest.cpp:77
+  verifyFormat("@property(a, b, c) int p;", "@property(b, a, c) int p;", 
Style);
+  verifyFormat("@property(a, b, c) int p;", "@property(c, b, a) int p;", 
Style);
+}
----------------
might it be a good idea to actually test some/all of the keywords you intend to 
support? just incase some of them are not actually tok::identifiers?


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

https://reviews.llvm.org/D150083

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

Reply via email to