[PATCH] D19482: [include-fixer] Add a find-all-symbols tool for include-fixer.

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 27 02:17:51 PDT 2016


hokein added inline comments.

================
Comment at: include-fixer/find-all-symbols/FindAllSymbols.h:38
@@ +37,3 @@
+private:
+  ResultReporter *const Reporter;
+};
----------------
bkramer wrote:
> nit: move const to the start of the line.
Moving `const` to the start of line will change  semantic of the declaration. We need a constant pointer here, not a pointer to a constant data.


http://reviews.llvm.org/D19482





More information about the cfe-commits mailing list