diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-01-19 09:13:03 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-01-19 13:26:27 +0100 |
| commit | be8c414567f49242164b1fdfb12764b16be355c1 (patch) | |
| tree | c1f505272ec27e5e069f5d6964dc4f7b20150a5a /compilerplugins | |
| parent | be94207ecb88a9005ee6624e354d70c9613d7653 (diff) | |
loplugin:unusedmethods also check for functions returning bool
we were previously excluding them
Change-Id: I48a68799b0de60b4995fae541eb363e043d4dd11
Reviewed-on: https://gerrit.libreoffice.org/48167
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'compilerplugins')
| -rwxr-xr-x | compilerplugins/clang/unusedmethods.py | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/compilerplugins/clang/unusedmethods.py b/compilerplugins/clang/unusedmethods.py index 811eca65cae4..58521e6b45b3 100755 --- a/compilerplugins/clang/unusedmethods.py +++ b/compilerplugins/clang/unusedmethods.py @@ -209,9 +209,6 @@ for d in definitionSet: # ignore methods with no return type, and constructors if d[0] == "void" or d[0] == "": 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 |