[PATCH] D26137: [clang-tidy] Add check name to YAML export

2016-11-22 Thread Alpha Abdoulaye via cfe-commits
Alpha added a comment. Ping Repository: rL LLVM https://reviews.llvm.org/D26137 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D26137: [clang-tidy] Add check name to YAML export

2016-11-22 Thread Alpha Abdoulaye via cfe-commits
Alpha added a comment. This shouldn't affect diagnostics without fixes. If there is no fix, there won't be anything to export, and the diagnostic just behaves normally. Repository: rL LLVM https://reviews.llvm.org/D26137 ___ cfe-commits mailing

[PATCH] D26137: [clang-tidy] Add check name to YAML export

2016-11-22 Thread Alpha Abdoulaye via cfe-commits
Alpha added a comment. In https://reviews.llvm.org/D26137#602602, @malcolm.parsons wrote: > In https://reviews.llvm.org/D26137#602591, @Alpha wrote: > > > This shouldn't affect diagnostics without fixes. If there is no fix, there > > won't be anything to export, and the diagnostic just behaves n

[PATCH] D26137: [clang-tidy] Add check name to YAML export

2016-10-31 Thread Alpha Abdoulaye via cfe-commits
Alpha created this revision. Alpha added reviewers: alexfh, klimek, djasper. Alpha added a subscriber: cfe-commits. Herald added subscribers: fhahn, mgorny. Add a field indicating the associated check for every replacement to the YAML report generated with the '-export-fixes' option. Update clang

[PATCH] D26137: [clang-tidy] Add check name to YAML export

2016-11-03 Thread Alpha Abdoulaye via cfe-commits
Alpha removed rL LLVM as the repository for this revision. Alpha updated this revision to Diff 76849. Alpha added a comment. Fix diagnostic deserialization bug for clang-apply-replacement. https://reviews.llvm.org/D26137 Files: include/clang/Tooling/Core/Diagnostic.h include/clang/Tooling/D

[PATCH] D26137: [clang-tidy] Add check name to YAML export

2016-11-03 Thread Alpha Abdoulaye via cfe-commits
Alpha updated this revision to Diff 76850. Alpha added a comment. Remove debug symbols. Repository: rL LLVM https://reviews.llvm.org/D26137 Files: include/clang/Tooling/Core/Diagnostic.h include/clang/Tooling/DiagnosticsYaml.h lib/Tooling/Core/CMakeLists.txt lib/Tooling/Core/Diagnost

[PATCH] D26137: [clang-tidy] Add check name to YAML export

2016-11-07 Thread Alpha Abdoulaye via cfe-commits
Alpha added a comment. Ping Repository: rL LLVM https://reviews.llvm.org/D26137 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D26137: [clang-tidy] Add check name to YAML export

2016-11-07 Thread Alpha Abdoulaye via cfe-commits
Alpha updated this revision to Diff 77029. Alpha added a comment. Ignore export of empty fixes. Repository: rL LLVM https://reviews.llvm.org/D26137 Files: include/clang/Tooling/Core/Diagnostic.h include/clang/Tooling/DiagnosticsYaml.h lib/Tooling/Core/CMakeLists.txt lib/Tooling/Core/

[PATCH] D26137: [clang-tidy] Add check name to YAML export

2016-11-08 Thread Alpha Abdoulaye via cfe-commits
Alpha added inline comments. Comment at: include/clang/Tooling/Core/Diagnostic.h:35 + DiagnosticMessage(llvm::StringRef Message, const SourceManager &Sources, +SourceLocation Loc); + std::string Message; alexfh wrote: > What are the constrai

[PATCH] D26137: [clang-tidy] Add check name to YAML export

2016-11-08 Thread Alpha Abdoulaye via cfe-commits
Alpha updated this revision to Diff 77196. Alpha added a comment. Export effectively MainSourceFile. Change CheckName field. Add doxygen-style comments. Repository: rL LLVM https://reviews.llvm.org/D26137 Files: include/clang/Tooling/Core/Diagnostic.h include/clang/Tooling/DiagnosticsYam

[PATCH] D26137: [clang-tidy] Add check name to YAML export

2016-11-14 Thread Alpha Abdoulaye via cfe-commits
Alpha added a comment. Ping Repository: rL LLVM https://reviews.llvm.org/D26137 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits