Szelethus added a comment.

Please don't mind me intruding.
If you create a patch for the Clang Static Analyzer, please add "[analyzer]" in 
the revision title, because many of us are automatically subscribed to those 
patches. Also, upload patches with full context.
https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface

I see the point here, and I think it'd be worth to just clang-format most of 
the files we work with, as the benefits would outweigh the drawbacks (at least 
eventually...), but I vaguely remember @george.karpenkov having many great 
points against it -- please don't commit until he can take a look :)


Repository:
  rC Clang

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

https://reviews.llvm.org/D54918



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

Reply via email to