Fixed some cppcheck messages

cli_ure/source/uno_bridge/cli_proxy.cxx

@@ -497,7 +497,6 @@ srrm::IMessage* UnoInterfaceProxy::Invoke(srrm::IMessage* callmsg)
                         method_td->pParams, args, argTypes, &pExc);
                     return constructReturnMessage(cli_ret, args, method_td,
                                                   callmsg, pExc);
-                    break;
                 }
             }
             else

extensions/source/ole/unoobjw.cxx

@@ -1007,7 +1007,7 @@ HRESULT InterfaceOleWrapper_Impl::doGetProperty( DISPPARAMS * /*pdispparams*/, V
         if (pvarResult)
             anyToVariant(pvarResult, returnValue);
     }
-    catch(const UnknownPropertyException e) //XInvocation::getValue
+    catch(const UnknownPropertyException& e) //XInvocation::getValue
     {
         writeExcepinfo(pexcepinfo, e.Message);
         ret = DISP_E_MEMBERNOTFOUND;

hwpfilter/source/hbox.cxx

@@ -335,12 +335,12 @@ TxtBox::~TxtBox(void)
     for (int ii = 0; ii < nCell; ++ii)
     {
         LinkedListIterator < HWPPara > it(&plists[ii]);
-        for (; it.current(); it++)
+        for (; it.current(); ++it)
             delete it.current();
     }
 
     LinkedListIterator < HWPPara > it(&caption);
-    for (; it.current(); it++)
+    for (; it.current(); ++it)
         delete it.current();
 
     delete[]plists;
@@ -369,7 +369,7 @@ Picture::~Picture(void)
         delete (HWPDrawingObject *) picinfo.picdraw.hdo;
 
     LinkedListIterator < HWPPara > it(&caption);
-    for (; it.current(); it++)
+    for (; it.current(); ++it)
         delete it.current();
 }
 
@@ -391,7 +391,7 @@ hunit Picture::Height(CharShape * sty)
 Hidden::~Hidden(void)
 {
     LinkedListIterator < HWPPara > it(&plist);
-    for (; it.current(); it++)
+    for (; it.current(); ++it)
         delete it.current();
 }
 
@@ -400,7 +400,7 @@ Hidden::~Hidden(void)
 HeaderFooter::~HeaderFooter(void)
 {
     LinkedListIterator < HWPPara > it(&plist);
-    for (; it.current(); it++)
+    for (; it.current(); ++it)
         delete it.current();
 }
 
@@ -409,7 +409,7 @@ HeaderFooter::~HeaderFooter(void)
 Footnote::~Footnote(void)
 {
     LinkedListIterator < HWPPara > it(&plist);
-    for (; it.current(); it++)
+    for (; it.current(); ++it)
         delete it.current();
 }
 

hwpfilter/source/hbox.h

@@ -537,7 +537,7 @@ struct Table
      Table(){};
      ~Table(){
           LinkedListIterator<TCell> it(&cells);
-          for( ; it.current(); it++)
+          for( ; it.current(); ++it)
                 delete it.current();
      };
      Columns columns;

hwpfilter/source/hwpfile.cxx

@@ -76,19 +76,19 @@ HWPFile::~HWPFile()
     delete hiodev;
 
     LinkedListIterator < ColumnInfo > it_column(&columnlist);
-    for (; it_column.current(); it_column++)
+    for (; it_column.current(); ++it_column)
         delete it_column.current();
 
     LinkedListIterator < HWPPara > it(&plist);
-    for (; it.current(); it++)
+    for (; it.current(); ++it)
         delete it.current();
 
     LinkedListIterator < Table > tbl(&tables);
-    for (; tbl.current(); tbl++)
+    for (; tbl.current(); ++tbl)
         delete tbl.current();
 
     LinkedListIterator < HyperText > hyp(&hyperlist);
-    for (; hyp.current(); hyp++)
+    for (; hyp.current(); ++hyp)
     {
         delete hyp.current();
     }
@@ -390,7 +390,7 @@ int HWPFile::GetPageMasterNum(int page)
     ColumnInfo *now = 0;
     int i;
 
-    for( i = 1 ; it.current() ; it++, i++){
+    for( i = 1 ; it.current() ; ++it, i++){
         now = it.current();
         if( page < now->start_page )
             return i-1;
@@ -412,7 +412,7 @@ EmPicture *HWPFile::GetEmPicture(Picture * pic)
     name[2] = 'P';
 
     LinkedListIterator < EmPicture > it(&emblist);
-    for (; it.current(); it++)
+    for (; it.current(); ++it)
         if (strcmp(name, it.current()->name) == 0)
             return it.current();
     return 0;
@@ -425,7 +425,7 @@ EmPicture *HWPFile::GetEmPictureByName(char * name)
     name[2] = 'P';
 
     LinkedListIterator < EmPicture > it(&emblist);
-    for (; it.current(); it++)
+    for (; it.current(); ++it)
         if (strcmp(name, it.current()->name) == 0)
             return it.current();
     return 0;