So, it's a small patch that I send for review because I do not know if
it's ok to do what the cppcheck says to do. In this case, he says that
for non-primitive types we can do ++test instead test++. My fear is if
in the code really need to be test++, understand?
revol_
diff --git a/sw/source/ui/index/cntex.cxx b/sw/source/ui/index/cntex.cxx
index 2b858e3..bcca510 100644
--- a/sw/source/ui/index/cntex.cxx
+++ b/sw/source/ui/index/cntex.cxx
@@ -423,7 +423,7 @@ void SwMultiTOXTabDialog::CreateOrUpdateExample(
pValues[nTokenIndex] = aPropVals;
nTokenIndex++;
- aIt++; // #i24377#
+ ++aIt; // #i24377#
}
aSequPropVals.realloc(nTokenIndex);
diff --git a/sw/source/ui/index/cnttab.cxx b/sw/source/ui/index/cnttab.cxx
index 24bd655..4b2b4c9 100644
--- a/sw/source/ui/index/cnttab.cxx
+++ b/sw/source/ui/index/cnttab.cxx
@@ -2497,7 +2497,7 @@ IMPL_LINK(SwTOXEntryTabPage, LevelHdl, SvTreeListBox*,
pBox)
aAuthFieldsLB.RemoveEntry(nLstBoxPos);
}
- aIt++; // #i21237#
+ ++aIt; // #i21237#
}
aAuthFieldsLB.SelectEntryPos(0);
}
@@ -2916,7 +2916,7 @@ void SwTokenWindow::SetForm(SwForm& rForm,
sal_uInt16 nL)
bLastWasText = sal_False;
}
- aIt++; // #i21237#
+ ++aIt; // #i21237#
}
if(!bLastWasText)
{
diff --git a/sw/source/ui/misc/redlndlg.cxx b/sw/source/ui/misc/redlndlg.cxx
index 0169751..2752bbf 100644
--- a/sw/source/ui/misc/redlndlg.cxx
+++ b/sw/source/ui/misc/redlndlg.cxx
@@ -861,7 +861,7 @@ void SwRedlineAcceptDlg::CallAcceptReject( BOOL bSelect,
BOOL bAccept )
ListBoxEntries_t::iterator aEnd = aRedlines.end();
for( ListBoxEntries_t::iterator aIter = aRedlines.begin();
aIter != aEnd;
- aIter++ )
+ ++aIter )
{
USHORT nPosition = GetRedlinePos( **aIter );
if( nPosition != USHRT_MAX )
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
