diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-12-20 15:53:21 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-12-21 08:22:29 +0100 |
| commit | 783120996aba341519393dd0100520707249d208 (patch) | |
| tree | aa6ff11749e3880ca7df05010277fde6a6269b45 /svl/source | |
| parent | 24158311c115c2db6dd05a751f75a5c084e2c0d1 (diff) | |
new loplugin: convertlong
merge the droplong and convertuintptr into one new plugin.
Limit the analysis to looking at var decl's, since that seems to be
safest proposition, even if that too needs some careful analysis.
Change-Id: Id005baaf05cfb157ce44a06a1c81f08559a07d1f
Reviewed-on: https://gerrit.libreoffice.org/46851
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'svl/source')
| -rw-r--r-- | svl/source/items/itemset.cxx | 2 | ||||
| -rw-r--r-- | svl/source/numbers/zforscan.cxx | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/svl/source/items/itemset.cxx b/svl/source/items/itemset.cxx index f520b26a7371..7e99dfa098fb 100644 --- a/svl/source/items/itemset.cxx +++ b/svl/source/items/itemset.cxx @@ -708,7 +708,7 @@ void SfxItemSet::SetRanges( const sal_uInt16 *pNewRanges ) } // create new item-array (by iterating through all new ranges) - sal_uLong nSize = Capacity_Impl(pNewRanges); + sal_uInt16 nSize = Capacity_Impl(pNewRanges); SfxItemArray aNewItems = new const SfxPoolItem* [ nSize ]; sal_uInt16 nNewCount = 0; if (m_nCount == 0) diff --git a/svl/source/numbers/zforscan.cxx b/svl/source/numbers/zforscan.cxx index 8f0a7b71f9ac..e47623228814 100644 --- a/svl/source/numbers/zforscan.cxx +++ b/svl/source/numbers/zforscan.cxx @@ -627,7 +627,7 @@ Color* ImpSvNumberformatScan::GetColor(OUString& sStr) if ( CharClass::isAsciiNumeric( sString ) ) { - long nIndex = sString.toInt32(); + sal_Int32 nIndex = sString.toInt32(); if (nIndex > 0 && nIndex <= 64) { pResult = pFormatter->GetUserDefColor((sal_uInt16)nIndex-1); |