krasimir added inline comments.

================
Comment at: lib/Format/Format.cpp:903-907
+  for (int i = Styles.size() - 1; i >= 0; --i) {
+    if (Styles[i].Language != FormatStyle::LK_None) {
+      Style->AddLanguageStyle(Styles[i]);
     }
   }
----------------
klimek wrote:
> This seems a bit backwards. I'd have expected us to build up a Style 
> structure after parsing above, and then just copy that into the result in the 
> end.
Do you mean to build up a StyleSet structure?


Repository:
  rC Clang

https://reviews.llvm.org/D41487



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

Reply via email to