diff options
author | Luboš Luňák <l.lunak@collabora.com> | 2019-10-06 14:22:59 +0200 |
---|---|---|
committer | Luboš Luňák <l.lunak@collabora.com> | 2019-10-07 21:56:55 +0200 |
commit | b1c14c30ba99ede85e41934b565195b0cbc69703 (patch) | |
tree | bf97cb604fc9704f60fcd2f480a9b2420b4ea604 /compilerplugins/clang/stringconstant.cxx | |
parent | 0ec6797f8063a1a82dc492840351c3f0f58088a4 (diff) |
fix various warnings in compilerplugins
These are triggered when using llvm-config --cxxflags for building,
and sometimes there's -Werror. The warnings were mostly unused
variables because of being used only in assert(), or default case
in switch that covers all enums (it's better to not handle default
to get warning if a case is not handled).
Change-Id: I0ecdd1f27390aadf033852b0d1ee0ca424ae3c37
Reviewed-on: https://gerrit.libreoffice.org/80317
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
Diffstat (limited to 'compilerplugins/clang/stringconstant.cxx')
-rw-r--r-- | compilerplugins/clang/stringconstant.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/compilerplugins/clang/stringconstant.cxx b/compilerplugins/clang/stringconstant.cxx index 8a56e8998b08..ed2fd52b75b0 100644 --- a/compilerplugins/clang/stringconstant.cxx +++ b/compilerplugins/clang/stringconstant.cxx @@ -1333,9 +1333,8 @@ std::string StringConstant::describeChangeKind(ChangeKind kind) { return "sal_Unicode argument"; case ChangeKind::OUStringLiteral1: return "OUStringLiteral1 argument"; - default: - std::abort(); } + llvm_unreachable("unknown change kind"); } bool StringConstant::isStringConstant( |