george.karpenkov added inline comments.

================
Comment at: lib/StaticAnalyzer/Core/RegionStore.cpp:1720
+    // Either the record variable or the field has to be const qualified.
+    if (RecordVarTy.isConstQualified() || Ty.isConstQualified()) {
+      if (const Expr *Init = VD->getInit()) {
----------------
style note -- by LLVM coding standards `{`'s are usually omitted when there's 
only a single statement inside the guard. Would help to avoid giant towers of 
`}`'s


Repository:
  rC Clang

https://reviews.llvm.org/D45774



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

Reply via email to