From 4c4999d944bbf8a34f8a320b8de6f4a325bd5d97 Mon Sep 17 00:00:00 2001 From: Jan Holesovsky Date: Tue, 8 Dec 2015 12:07:49 +0100 Subject: 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 --- helpcompiler/source/HelpCompiler.cxx | 4 ++-- helpcompiler/source/HelpLinker.cxx | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) (limited to 'helpcompiler') 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 fb7425e3dd03..73ba6a3a2919 100644 --- a/helpcompiler/source/HelpLinker.cxx +++ b/helpcompiler/source/HelpLinker.cxx @@ -396,7 +396,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"; -- cgit