diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-08-14 08:28:13 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-08-14 09:14:39 +0200 |
| commit | 0c3bd1d482ece1f2217cc87069179161f6a5bb56 (patch) | |
| tree | 86bab8ab624fd60076838f068f010372321a204b /comphelper | |
| parent | 11535357df4e4ec828435229ff784b9868ee583a (diff) | |
new loplugin:droplong
The types 'long' and 'unsigned long' are different sizes on different
platforms, making them wholy unsuitable for portable code.
And when I mean different sizes, I mean 64bit Linux and 64bit Windows
have different bit sizes.
Change-Id: Id4d8cd709bf71a3f85ed6515fae2b0a21ae8c25d
Reviewed-on: https://gerrit.libreoffice.org/41130
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'comphelper')
| -rw-r--r-- | comphelper/source/misc/backupfilehelper.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/comphelper/source/misc/backupfilehelper.cxx b/comphelper/source/misc/backupfilehelper.cxx index ebf93aeb8282..abc8579a542b 100644 --- a/comphelper/source/misc/backupfilehelper.cxx +++ b/comphelper/source/misc/backupfilehelper.cxx @@ -654,9 +654,9 @@ namespace if (aList.is()) { - const long nLength(aList->getLength()); + const sal_Int32 nLength(aList->getLength()); - for (long a(0); a < nLength; a++) + for (sal_Int32 a(0); a < nLength; a++) { const uno::Reference< xml::dom::XElement > aChild(aList->item(a), uno::UNO_QUERY); @@ -765,9 +765,9 @@ namespace if (aList.is()) { - const long nLength(aList->getLength()); + const sal_Int32 nLength(aList->getLength()); - for (long a(0); a < nLength; a++) + for (sal_Int32 a(0); a < nLength; a++) { const uno::Reference< xml::dom::XElement > aChild(aList->item(a), uno::UNO_QUERY); |