djasper added inline comments.

================
Comment at: lib/Format/TokenAnnotator.cpp:82
         CurrentToken->MatchingParen = Left;
-        CurrentToken->Type = TT_TemplateCloser;
+        if (Style.Language == FormatStyle::LK_TextProto)
+          CurrentToken->Type = TT_DictLiteral;
----------------
Here and in many other places, you are only checking against LK_TextProto. 
However, in LK_Proto, we also have a lot of text-formatted protos in field 
options. So, most places, these two should result in the same behavior, I think.


================
Comment at: lib/Format/TokenAnnotator.cpp:2517
+  if (Left.is(tok::greater) && Right.is(tok::greater)) {
+    if (Style.Language == FormatStyle::LK_TextProto) {
+      return !Style.Cpp11BracedListStyle;
----------------
No braces.


Repository:
  rC Clang

https://reviews.llvm.org/D42727



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

Reply via email to