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

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 8 08:20:29 PST 2016


alexfh requested changes to this revision.
alexfh added a comment.
This revision now requires changes to proceed.

Looks like compiler has found a couple of bugs:

  In file included from llvm.git/tools/clang/tools/extra/clang-tidy/ClangTidy.cpp:38:
  llvm.git/tools/clang/include/clang/Tooling/DiagnosticsYaml.h:38:53: error: field 'Message' is uninitialized when used here [-Werror,-Wuninitialized]
          : DiagnosticName(D.DiagnosticName), Message(Message), Fix(D.Fix),
                                                      ^
  llvm.git/tools/clang/include/clang/Tooling/DiagnosticsYaml.h:70:7: error: ignoring return value of function declared with warn_unused_result attribute [-Werror,-Wunused-result]
        Keys->Fix[Fix.getFilePath()].add(Fix);
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~
  2 errors generated.

Please fix.


Repository:
  rL LLVM

https://reviews.llvm.org/D26137





More information about the cfe-commits mailing list