[PATCH] Don't break string literals inside the _T() macro. A workaround for http://llvm.org/PR17122

Daniel Jasper djasper at google.com
Tue Sep 10 07:52:28 PDT 2013


  I don't see urgency in submitting this instead of just implementing the splitting. Or is this going to take very long?


================
Comment at: lib/Format/ContinuationIndenter.cpp:661
@@ +660,3 @@
+        Current.Previous->Previous &&
+        Current.Previous->Previous->TokenText == "_T" && Current.Next &&
+        Current.Next->is(tok::r_paren))
----------------
How is the next token being an r_paren relevant? I there something that can break without it?


http://llvm-reviews.chandlerc.com/D1640



More information about the cfe-commits mailing list