[PATCH] D10305: [Clang Static Analyzer] Bug identification

Gábor Horváth xazax.hun at gmail.com
Mon Jul 13 13:52:01 PDT 2015


xazax.hun added a comment.

Some minor style comments.


================
Comment at: include/clang/StaticAnalyzer/Core/BugReporter/BugReporter.h:146
@@ -144,2 +145,3 @@
 
 public:
+  BugReport(BugType &bt, StringRef desc, const ExplodedNode *errornode,
----------------
Ampersand should be next to the variable name. On some other places too.

================
Comment at: lib/StaticAnalyzer/Core/BugId.cpp:2
@@ +1,3 @@
+//===---------- BugId.cpp - Unique bugid utility ------------------*- C++
+//-*-===//
+//
----------------
The top of the header should be one line and should be within the 80 column limit.


http://reviews.llvm.org/D10305







More information about the cfe-commits mailing list