new loplugin: commaoperator

Change-Id: I03f24e61f696b7619855e3c7010aa0d874e5a4ff

3 files changed, 17 insertions, 7 deletions

diff --git a/i18npool/source/defaultnumberingprovider/defaultnumberingprovider.cxx b/i18npool/source/defaultnumberingprovider/defaultnumberingprovider.cxx
index 40fb536e848a..aa7ad1d208cb 100644
--- a/i18npool/source/defaultnumberingprovider/defaultnumberingprovider.cxx
+++ b/i18npool/source/defaultnumberingprovider/defaultnumberingprovider.cxx
@@ -979,7 +979,7 @@ DefaultNumberingProvider::isScriptFlagEnabled(const OUString& aName) throw(Runti
         Sequence< Any > aArgs(1);
         beans::PropertyValue aPath;
         aPath.Name = "nodepath";
-        aPath.Value <<= OUString("/org.openoffice.Office.Common/I18N"),
+        aPath.Value <<= OUString("/org.openoffice.Office.Common/I18N");
         aArgs[0] <<= aPath;
 
         Reference<XInterface> xInterface = xConfigProvider->createInstanceWithArguments(
diff --git a/i18npool/source/localedata/LocaleNode.cxx b/i18npool/source/localedata/LocaleNode.cxx
index 098c39228fc0..a811fb211f64 100644
--- a/i18npool/source/localedata/LocaleNode.cxx
+++ b/i18npool/source/localedata/LocaleNode.cxx
@@ -452,11 +452,17 @@ void LCCTYPENode::generateCode (const OFileWriter &of) const
         ++nSavErr;  // format codes not affected
     }
     if (aTimeSep == aTime100Sep)
-        ++nWarn, fprintf( stderr, "Warning: %s\n",
+    {
+        ++nWarn;
+        fprintf( stderr, "Warning: %s\n",
                 "Time100SecSeparator equals TimeSeparator, this is probably an error.");
+    }
     if (aDecSep != aTime100Sep)
-        ++nWarn, fprintf( stderr, "Warning: %s\n",
+    {
+        ++nWarn;
+        fprintf( stderr, "Warning: %s\n",
                 "Time100SecSeparator is different from DecimalSeparator, this may be correct or not. Intended?");
+    }
     if (nSavErr != nError || nWarn)
         fprintf( stderr, "Warning: %s\n",
                 "Don't forget to adapt corresponding FormatCode elements when changing separators.");
diff --git a/i18npool/source/search/textsearch.cxx b/i18npool/source/search/textsearch.cxx
index 90bced1be8f8..9d000021c0fd 100644
--- a/i18npool/source/search/textsearch.cxx
+++ b/i18npool/source/search/textsearch.cxx
@@ -136,10 +136,14 @@ void TextSearch::setOptions2( const SearchOptions2& rOptions ) throw( RuntimeExc
 {
     aSrchPara = rOptions;
 
-    delete pRegexMatcher, pRegexMatcher = nullptr;
-    delete pWLD, pWLD = nullptr;
-    delete pJumpTable, pJumpTable = nullptr;
-    delete pJumpTable2, pJumpTable2 = nullptr;
+    delete pRegexMatcher;
+    pRegexMatcher = nullptr;
+    delete pWLD;
+    pWLD = nullptr;
+    delete pJumpTable;
+    pJumpTable = nullptr;
+    delete pJumpTable2;
+    pJumpTable2 = nullptr;
     maWildcardReversePattern.clear();
     maWildcardReversePattern2.clear();