diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2017-03-25 10:55:44 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2017-03-25 16:21:34 +0000 |
| commit | 2258f33a5fd95a5e25f5bb232994ab147d09bfb9 (patch) | |
| tree | 32f39a1ebbd79bb74e6dac67cb018c7a3c313a5e /solenv | |
| parent | 5d8e6901ec14edd9da10d9dee63b2ef2ab058692 (diff) | |
Make loplugin:loopvartoosmall find more suspicious cases
...where the "controlling expression" of any sort of loop contains a sub-
expression of the form
var < val
where the type of var is smaller than that of val. Theoretically, this could
turn up lots of false positives, but practically it didn't run into any. Most
findings have been cleaned up over the last weeks. There's just a handful
remaining places that are hard to clean up, so I flagged them here with
(deliberately awkward) sal::static_int_cast for later clean-up.
Change-Id: I0f735d46dda15b9b336150095df65cf247e9d6d3
Reviewed-on: https://gerrit.libreoffice.org/35682
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'solenv')
| -rw-r--r-- | solenv/CompilerTest_compilerplugins_clang.mk | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/solenv/CompilerTest_compilerplugins_clang.mk b/solenv/CompilerTest_compilerplugins_clang.mk index 80508e8ce6a5..724a9ac07046 100644 --- a/solenv/CompilerTest_compilerplugins_clang.mk +++ b/solenv/CompilerTest_compilerplugins_clang.mk @@ -13,6 +13,7 @@ $(eval $(call gb_CompilerTest_add_exception_objects,compilerplugins_clang, \ compilerplugins/clang/test/datamembershadow \ compilerplugins/clang/test/externvar \ compilerplugins/clang/test/finalprotected \ + compilerplugins/clang/test/loopvartoosmall \ compilerplugins/clang/test/passstuffbyref \ compilerplugins/clang/test/oslendian-1 \ compilerplugins/clang/test/oslendian-2 \ |