Hi,
I have submitted a patch for review:
https://gerrit.libreoffice.org/3868
To pull it, you can do:
git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/68/3868/1
cppcheck: fix arrayIndexThenCheck in vcl
Change-Id: Id7258dd35c131c2c8151bc5e0f97076241e6d10f
---
M vcl/source/control/field.cxx
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/vcl/source/control/field.cxx b/vcl/source/control/field.cxx
index c035678..87043dd 100644
--- a/vcl/source/control/field.cxx
+++ b/vcl/source/control/field.cxx
@@ -201,7 +201,7 @@
{
// check if string is equivalent to zero
sal_Int16 nIndex = bNegative ? 1 : 0;
- while( aStr[nIndex] == '0' && nIndex < aStr.getLength() )
+ while (nIndex < aStr.getLength() && aStr[nIndex] == '0')
++nIndex;
if( nIndex < aStr.getLength() )
{
--
To view, visit https://gerrit.libreoffice.org/3868
To unsubscribe, visit https://gerrit.libreoffice.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id7258dd35c131c2c8151bc5e0f97076241e6d10f
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Julien Nabet <serval2...@yahoo.fr>
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
