bkramer added inline comments. ================ Comment at: include-fixer/IncludeFixer.h:71 @@ +70,3 @@ +/// \param FirstIncludeOffset The insertion point for new include directives. +/// -1U(UINT_MAX) indicates uninitialization, which will insert the header at +/// first line. If there is no #include block in the Code, it will create a ---------------- I think you can drop the part about 'uninitialization' from this comment.
================ Comment at: include-fixer/IncludeFixer.h:79 @@ +78,3 @@ + StringRef Code, StringRef FilePath, StringRef Header, + unsigned FirstIncludeOffset=-1U, + const clang::format::FormatStyle &Style=clang::format::getLLVMStyle()); ---------------- This doesn't look clang-formatted :p http://reviews.llvm.org/D20621 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
