diff options
author | Noel Grandin <noel@peralex.com> | 2015-05-27 16:33:44 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2015-05-28 12:49:54 +0200 |
commit | d5129a9dd68978f9eccdd4597b5b6834557c422a (patch) | |
tree | df43250172f784f3048ce42ce1c3410d1d449c1e /vcl/generic/print/prtsetup.cxx | |
parent | convert PARENTCLIPMODE constants to scoped enum (diff) | |
download | core-d5129a9dd68978f9eccdd4597b5b6834557c422a.tar.gz core-d5129a9dd68978f9eccdd4597b5b6834557c422a.zip |
new clang plugin: loopvartoosmall
Idea from bubli - look for loops where the index variable is of such
size that it cannot cover the range revealed by examining the length
part of the condition.
So far, I have only run the plugin up till the VCL module.
Also the plugin deliberately excludes anything more complicated than a
straightforward incrementing for loop.
Change-Id: Ifced18b01c03ea537c64168465ce0b8287a42015
Diffstat (limited to 'vcl/generic/print/prtsetup.cxx')
-rw-r--r-- | vcl/generic/print/prtsetup.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/vcl/generic/print/prtsetup.cxx b/vcl/generic/print/prtsetup.cxx index 738d00de721a..9d74f048ae3a 100644 --- a/vcl/generic/print/prtsetup.cxx +++ b/vcl/generic/print/prtsetup.cxx @@ -341,7 +341,7 @@ RTSDevicePage::RTSDevicePage( RTSDialog* pParent ) m_pLevelBox->SetEntryData(0, m_pLevelBox->GetEntryData(1)); m_pLevelBox->RemoveEntry(1); - for( sal_uInt16 i = 0; i < m_pLevelBox->GetEntryCount(); i++ ) + for( sal_Int32 i = 0; i < m_pLevelBox->GetEntryCount(); i++ ) { if( reinterpret_cast<sal_uLong>(m_pLevelBox->GetEntryData( i )) == nLevelEntryData ) { |