bin/find-unused-defines.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
New commits: commit 5e80aa818cae690c67361e85bd37124cec16c05e Author: LeSasse <[email protected]> AuthorDate: Wed Mar 27 09:50:53 2024 +0100 Commit: Noel Grandin <[email protected]> CommitDate: Thu Mar 28 08:49:48 2024 +0100 tdf#158803 test for membership should be 'not in', unnecessary semicolons Change-Id: If468514ce934eb5a583630b2fbe8634a77a5ff94 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165379 Reviewed-by: Noel Grandin <[email protected]> Tested-by: Noel Grandin <[email protected]> diff --git a/bin/find-unused-defines.py b/bin/find-unused-defines.py index 53db9d1d532a..25a50af04962 100755 --- a/bin/find-unused-defines.py +++ b/bin/find-unused-defines.py @@ -95,8 +95,8 @@ exclusionSet = set([ def in_exclusion_set( a ): for f in exclusionSet: if a.startswith(f): - return True; - return False; + return True + return False # Hack to turn off unicode decoding errors, which sometimes happens in the output and I really don't @@ -137,9 +137,9 @@ with a.stdout as txt: if line2.startswith("/*"): continue # check if we found one in actual code if idName.startswith("SID_"): - if not ".hrc:" in line2 and not ".src:" in line2 and not ".sdi:" in line2: found_reason_to_exclude = True + if ".hrc:" not in line2 and ".src:" not in line2 and ".sdi:" not in line2: found_reason_to_exclude = True else: - if not ".hrc:" in line2 and not ".src:" in line2: found_reason_to_exclude = True + if ".hrc:" not in line2 and ".src:" not in line2: found_reason_to_exclude = True if idName.startswith("RID_"): # is the constant being used as an identifier by entries in .src files? if ".src:" in line2 and "Identifier = " in line2: found_reason_to_exclude = True
