osmocom
/
wireshark
14
0
Fork 1
Code Issues Pull Requests Projects Releases Wiki Activity

check_dissector_urls.py: remove unnecessary wrapper 

The first parameter for filter must be a function that takes an element
and returns True or False. We can use is_dissector_file directly, there's
no need to wrap it into an anonymous function.
This commit is contained in:
Martin Kaiser 2023-04-01 16:32:56 +01:00 committed by Martin Mathieson
parent 8d98e6319c
commit fae0f31ff1
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tools/check_dissector_urls.py
View File

@ -200,7 +200,7 @@ elif args.commits:
files = [f.decode('utf-8')
for f in subprocess.check_output(command).splitlines()]
# Fetch links from files (dissectors files only)
files = list(filter(lambda f: is_dissector_file(f), files))
files = list(filter(is_dissector_file, files))
for f in files:
find_links_in_file(f)
elif args.open:
@ -208,12 +208,12 @@ elif args.open:
command = ['git', 'diff', '--name-only']
files = [f.decode('utf-8')
for f in subprocess.check_output(command).splitlines()]
files = list(filter(lambda f: is_dissector_file(f), files))
files = list(filter(is_dissector_file, files))
# Staged changes.
command = ['git', 'diff', '--staged', '--name-only']
files_staged = [f.decode('utf-8')
for f in subprocess.check_output(command).splitlines()]
files_staged = list(filter(lambda f: is_dissector_file(f), files_staged))
files_staged = list(filter(is_dissector_file, files_staged))
for f in files:
find_links_in_file(f)
for f in files_staged: