diff options
| author | Noel Grandin <noel@peralex.com> | 2016-01-20 16:36:19 +0200 |
|---|---|---|
| committer | Noel Grandin <noelgrandin@gmail.com> | 2016-01-21 07:18:38 +0000 |
| commit | 90d892664a6c49a8ae25a72ddccf54dba9d0e429 (patch) | |
| tree | d9bb4a7307e349b88a4d528854abb44e4323f563 /compilerplugins | |
| parent | ea6fdbaeeb51ad695f1c3754b796b7273eb4baad (diff) | |
loplugin: unused return values
Change-Id: I4eb1f0c9245c04058fd5e47046f043f8840a79c7
Reviewed-on: https://gerrit.libreoffice.org/21628
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'compilerplugins')
| -rwxr-xr-x | compilerplugins/clang/unusedmethods.py | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/compilerplugins/clang/unusedmethods.py b/compilerplugins/clang/unusedmethods.py index d5c7946da822..2f94f16bd9ca 100755 --- a/compilerplugins/clang/unusedmethods.py +++ b/compilerplugins/clang/unusedmethods.py @@ -241,10 +241,10 @@ def natural_sort_key(s, _nsre=re.compile('([0-9]+)')): tmp1list = sorted(tmp1set, key=lambda v: natural_sort_key(v[1])) # print out the results -for t in tmp1list: - print t[1] - print " ", t[0] -sys.exit(0) +#for t in tmp1list: +# print t[1] +# print " ", t[0] +#sys.exit(0) # ------------------------------------------- # Do the "unused return types" part @@ -261,6 +261,9 @@ for d in definitionSet: continue if d[0] == "void": continue + # ignore bool returns, provides important documentation in the code + if d[0] == "_Bool": + continue # ignore UNO constructor method entrypoints if "_get_implementation" in d[1] or "_getFactory" in d[1]: continue |