diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2015-06-25 15:59:01 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2015-06-25 15:59:01 +0200 |
commit | d4ad86dac6e8971a4f21168051876a0d68f48649 (patch) | |
tree | f2ba347d2c34a89c567850a7ddac3661463e08d8 /compilerplugins | |
parent | a8cd99685abf879c56e04a95a5b2881d2adddcb9 (diff) |
...in case this ever gets used with an unsigned APSInt of bitwidth < 7
Change-Id: Ic9436529321e523edd25d0f83ac09a0e22a44013
Diffstat (limited to 'compilerplugins')
-rw-r--r-- | compilerplugins/clang/stringconstant.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/compilerplugins/clang/stringconstant.cxx b/compilerplugins/clang/stringconstant.cxx index 20ff2a996cce..1280ce121691 100644 --- a/compilerplugins/clang/stringconstant.cxx +++ b/compilerplugins/clang/stringconstant.cxx @@ -1191,7 +1191,7 @@ void StringConstant::handleOUStringCtor( && e3->getArg(0)->IgnoreParenImpCasts()->isIntegerConstantExpr( res, compiler.getASTContext())) { - if (res.getSExtValue() > 0 && res.getSExtValue() <= 127) { + if (res.getZExtValue() > 0 && res.getZExtValue() <= 127) { report( DiagnosticsEngine::Warning, ("in call of %0, replace OUString constructed from a (non-NUL)" |