diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2018-08-13 09:05:19 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2018-08-13 12:44:31 +0200 |
commit | ab76a889713661030c5839e5cc04820726a31c38 (patch) | |
tree | c8b60aeb80999a6e6a153922aca20224f6ed3815 /compilerplugins | |
parent | fbc0524ee19e039cadea92d2dd163525bad5f661 (diff) |
Various loplugin warnings in mysql
I assume the "0 == " around columnStringValue.equalsIgnoreAsciiCase in
ODatabaseMetaData::getSchemas (mysqlc/source/mysqlc_databasemetadata.cxx) was
just a thinko, as the latter already returns sal_Bool, not a <0 / 0 / >0 integer
value.
(Rebased atop ce24919c01d1ab77f962137ff04a31dd5ec944af "loplugin, various in
mysqlc", which covered the same warnings, but didn't consider the negated call
to equalsIgnoreAsciiCase to be an accident as I do here (see above), and
accidentally reversed the logic of OResultSetMetaData::isNullable in
mysqlc/source/mysqlc_resultsetmetadata.cxx, I think.)
Change-Id: Ie89cc8ef7c7638d28307a4b240930d387d75b8f2
Reviewed-on: https://gerrit.libreoffice.org/58908
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'compilerplugins')
-rw-r--r-- | compilerplugins/clang/check.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/compilerplugins/clang/check.cxx b/compilerplugins/clang/check.cxx index b31e40b23e2b..86484d6828cf 100644 --- a/compilerplugins/clang/check.cxx +++ b/compilerplugins/clang/check.cxx @@ -89,7 +89,7 @@ TerminalCheck TypeCheck::AnyBoolean() const { n == "sal_Bool" || n == "BOOL" || n == "Boolean" || n == "FT_Bool" || n == "FcBool" || n == "GLboolean" || n == "NPBool" || n == "TW_BOOL" || n == "UBool" || n == "boolean" || n == "dbus_bool_t" - || n == "gboolean" || n == "hb_bool_t" || n == "jboolean"); + || n == "gboolean" || n == "hb_bool_t" || n == "jboolean" || n == "my_bool"); } TypeCheck TypeCheck::LvalueReference() const { |