diff options
author | Jan Holesovsky <kendy@collabora.com> | 2015-12-08 12:07:49 +0100 |
---|---|---|
committer | Jan Holesovsky <kendy@collabora.com> | 2015-12-08 12:23:55 +0100 |
commit | 858613fb2fb554c57081f54aea6a0e2782b79aaa (patch) | |
tree | bd050894f82c1b622f66c8e8d28dc4e39f7951be | |
parent | 05f49a7eb51b0c6c158d41f7c8bec6b51896e120 (diff) |
tdf#96042: 'std::string::find("something") == 0' means "startsWith()".
This should fix a regression from 3bdd176731c351638f541a37b94094124f3c9f52,
apparently the cppcheck's advice is misleading.
Change-Id: I427ecaa1eb3c9841cb6112997b9b51feda4583d0
-rw-r--r-- | compilerplugins/clang/passstuffbyref.cxx | 2 | ||||
-rw-r--r-- | helpcompiler/source/HelpCompiler.cxx | 4 | ||||
-rw-r--r-- | helpcompiler/source/HelpLinker.cxx | 2 |
3 files changed, 4 insertions, 4 deletions
diff --git a/compilerplugins/clang/passstuffbyref.cxx b/compilerplugins/clang/passstuffbyref.cxx index 673b72b32f8e..f2e66f710986 100644 --- a/compilerplugins/clang/passstuffbyref.cxx +++ b/compilerplugins/clang/passstuffbyref.cxx @@ -103,7 +103,7 @@ bool PassStuffByRef::isFat(QualType type, std::string * name) { } *name = type.getUnqualifiedType().getCanonicalType().getAsString(); if (*name == "class rtl::OUString" || *name == "class rtl::OString" - || name->compare("class com::sun::star::uno::Sequence") == 0) + || name->compare(0, 35, "class com::sun::star::uno::Sequence") == 0) { return true; } diff --git a/helpcompiler/source/HelpCompiler.cxx b/helpcompiler/source/HelpCompiler.cxx index 85a82d043bed..6d821e452b1a 100644 --- a/helpcompiler/source/HelpCompiler.cxx +++ b/helpcompiler/source/HelpCompiler.cxx @@ -334,7 +334,7 @@ void myparser::traverse( xmlNodePtr parentNode ) std::string hid; - if (branch.compare("hid") == 0) + if (branch.compare(0, 3, "hid") != 0) { size_t index = branch.find('/'); if (index != std::string::npos) @@ -492,7 +492,7 @@ bool HelpCompiler::compile() if ( !bExtensionMode && !fileName.empty()) { - if (fileName.compare("/text/") == 0) + if (fileName.compare(0, 6, "/text/") == 0) { int len = strlen("/text/"); actMod = fileName.substr(len); diff --git a/helpcompiler/source/HelpLinker.cxx b/helpcompiler/source/HelpLinker.cxx index 7d4cfe53b880..13b840d5508e 100644 --- a/helpcompiler/source/HelpLinker.cxx +++ b/helpcompiler/source/HelpLinker.cxx @@ -397,7 +397,7 @@ void HelpLinker::link() throw(HelpProcessingException, BasicCodeTagger::TaggerEx continue; std::string documentPath = streamTable.document_path; - if (documentPath.compare("/") == 0) + if (documentPath.compare(0, 1, "/") == 0) documentPath = documentPath.substr(1); std::string documentJarfile = streamTable.document_module + ".jar"; |