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 /l10ntools/source/xrmmerge.cxx | |
parent | f3331f7694e74f349375c223ce7ed84838e92d89 (diff) |
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 'l10ntools/source/xrmmerge.cxx')
-rw-r--r-- | l10ntools/source/xrmmerge.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/l10ntools/source/xrmmerge.cxx b/l10ntools/source/xrmmerge.cxx index 421195d54e10..b47c7c38dcdd 100644 --- a/l10ntools/source/xrmmerge.cxx +++ b/l10ntools/source/xrmmerge.cxx @@ -413,7 +413,7 @@ void XRMResMerge::WorkOnDesc( if ( pEntrys ) { OString sCur; OString sDescFilename = GetAttribute ( rOpenTag, "xlink:href" ); - for( unsigned int n = 0; n < aLanguages.size(); n++ ){ + for( size_t n = 0; n < aLanguages.size(); n++ ){ sCur = aLanguages[ n ]; OString sContent; if ( !sCur.equalsIgnoreAsciiCase("en-US") && @@ -502,7 +502,7 @@ void XRMResMerge::EndOfText( MergeEntrys *pEntrys = pMergeDataFile->GetMergeEntrys( pResData ); if ( pEntrys ) { OString sCur; - for( unsigned int n = 0; n < aLanguages.size(); n++ ){ + for( size_t n = 0; n < aLanguages.size(); n++ ){ sCur = aLanguages[ n ]; OString sContent; if (!sCur.equalsIgnoreAsciiCase("en-US") && |