[Libreoffice-commits] .: 2 commits - binfilter/bf_sw unoxml/source

Caolán McNamara caolan at kemper.freedesktop.org
Fri Nov 26 13:49:29 PST 2010


 binfilter/bf_sw/source/filter/xml/sw_XMLRedlineImportHelper.cxx |    2 -
 unoxml/source/dom/saxbuilder.cxx                                |   12 +++++-----
 unoxml/source/xpath/xpathapi.cxx                                |    4 +--
 3 files changed, 9 insertions(+), 9 deletions(-)

New commits:
commit b53b1e980ff83bd2b28ff709d35308cee079e75a
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Fri Nov 26 21:35:36 2010 +0000

    cppcheck: use prefix variant

diff --git a/unoxml/source/dom/saxbuilder.cxx b/unoxml/source/dom/saxbuilder.cxx
index 137ce87..17d7eac 100644
--- a/unoxml/source/dom/saxbuilder.cxx
+++ b/unoxml/source/dom/saxbuilder.cxx
@@ -263,10 +263,8 @@ namespace DOM
             attr_qname = a->first;
             attr_value = a->second;
             idx = attr_qname.indexOf(':');
-            if(idx != -1)
-            {
+            if (idx != -1)
                 aPrefix = attr_qname.copy(0, idx);
-            }
             else
                 aPrefix = OUString();
 
@@ -275,11 +273,13 @@ namespace DOM
             {
                 // set attribute with namespace
                 aElement->setAttributeNS(result->second, attr_qname, attr_value);
-            } else {
+            }
+            else
+            {
                 // set attribute without namespace
                 aElement->setAttribute(attr_qname, attr_value);
-           }
-            a++;
+            }
+            ++a;
         }
         m_aNSStack.push(aNSMap);
     }
diff --git a/unoxml/source/xpath/xpathapi.cxx b/unoxml/source/xpath/xpathapi.cxx
index cb88a51..97c5a88 100644
--- a/unoxml/source/xpath/xpathapi.cxx
+++ b/unoxml/source/xpath/xpathapi.cxx
@@ -134,7 +134,7 @@ namespace XPath
             p = (xmlChar*)oprefix.getStr();
             u = (xmlChar*)ouri.getStr();
             xmlXPathRegisterNs(ctx, p, u);
-            i++;
+            ++i;
         }
     }
 
@@ -185,7 +185,7 @@ namespace XPath
                     reinterpret_cast<void*>(
                         sal::static_int_cast<sal_IntPtr>(aHandle.variableData)));
             }
-            i++;
+            ++i;
         }
     }
 
commit 8599fb3e28c8a5ecc30919cd94510550ac980aa4
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Fri Nov 26 21:31:19 2010 +0000

    cppcheck: use prefix variant

diff --git a/binfilter/bf_sw/source/filter/xml/sw_XMLRedlineImportHelper.cxx b/binfilter/bf_sw/source/filter/xml/sw_XMLRedlineImportHelper.cxx
index 80162c5..159cdb8 100644
--- a/binfilter/bf_sw/source/filter/xml/sw_XMLRedlineImportHelper.cxx
+++ b/binfilter/bf_sw/source/filter/xml/sw_XMLRedlineImportHelper.cxx
@@ -332,7 +332,7 @@ XMLRedlineImportHelper::~XMLRedlineImportHelper()
 {
     // delete all left over (and obviously incomplete) RedlineInfos (and map)
     RedlineMapType::iterator aFind = aRedlineMap.begin();
-    for( ; aRedlineMap.end() != aFind; aFind++ )
+    for( ; aRedlineMap.end() != aFind; ++aFind )
     {
         RedlineInfo* pInfo = aFind->second;
 


More information about the Libreoffice-commits mailing list