diff options
author | Fangrui Song <maskray@google.com> | 2019-08-28 02:56:03 +0000 |
---|---|---|
committer | Fangrui Song <maskray@google.com> | 2019-08-28 02:56:03 +0000 |
commit | 180380651c5e6f64d69b541bb1543099d0f9f9c8 (patch) | |
tree | 7deaea5f1d7e609da057199998a090e805d51f48 /clang-tools-extra/clang-doc/Representation.cpp | |
parent | acd8499028fd774bed7b0a9bfe6d9802b5ae7daa (diff) | |
download | bcm5719-llvm-180380651c5e6f64d69b541bb1543099d0f9f9c8.tar.gz bcm5719-llvm-180380651c5e6f64d69b541bb1543099d0f9f9c8.zip |
[clang-doc] Use llvm::createStringError and canonicalize error messages
"Bad block found.\n" -> "bad block found"
The lower cased form with no full stop or newline is more common in LLVM
tools.
Reviewed By: juliehockett
Differential Revision: https://reviews.llvm.org/D66783
llvm-svn: 370155
Diffstat (limited to 'clang-tools-extra/clang-doc/Representation.cpp')
-rw-r--r-- | clang-tools-extra/clang-doc/Representation.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/clang-tools-extra/clang-doc/Representation.cpp b/clang-tools-extra/clang-doc/Representation.cpp index bf3b1f7ecef..29b6807e0bf 100644 --- a/clang-tools-extra/clang-doc/Representation.cpp +++ b/clang-tools-extra/clang-doc/Representation.cpp @@ -34,8 +34,8 @@ template <typename T> llvm::Expected<std::unique_ptr<Info>> reduce(std::vector<std::unique_ptr<Info>> &Values) { if (Values.empty()) - return llvm::make_error<llvm::StringError>(" No values to reduce.\n", - llvm::inconvertibleErrorCode()); + return llvm::createStringError(llvm::inconvertibleErrorCode(), + "no value to reduce"); std::unique_ptr<Info> Merged = std::make_unique<T>(Values[0]->USR); T *Tmp = static_cast<T *>(Merged.get()); for (auto &I : Values) @@ -96,8 +96,8 @@ void reduceChildren(std::vector<EnumInfo> &Children, llvm::Expected<std::unique_ptr<Info>> mergeInfos(std::vector<std::unique_ptr<Info>> &Values) { if (Values.empty()) - return llvm::make_error<llvm::StringError>("No info values to merge.\n", - llvm::inconvertibleErrorCode()); + return llvm::createStringError(llvm::inconvertibleErrorCode(), + "no info values to merge"); switch (Values[0]->IT) { case InfoType::IT_namespace: @@ -109,8 +109,8 @@ mergeInfos(std::vector<std::unique_ptr<Info>> &Values) { case InfoType::IT_function: return reduce<FunctionInfo>(Values); default: - return llvm::make_error<llvm::StringError>("Unexpected info type.\n", - llvm::inconvertibleErrorCode()); + return llvm::createStringError(llvm::inconvertibleErrorCode(), + "unexpected info type"); } } |