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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
