def-tool: Hoist loop invariant
This commit hoists loop invariant (collect_symbols()) out of the loop.
Test: ./tests/run.py
Change-Id: I865fb1a2da96d1fdecd3689795d6c46e42d4783c
diff --git a/vndk/tools/definition-tool/vndk_definition_tool.py b/vndk/tools/definition-tool/vndk_definition_tool.py
index 50a7b2b..d62fcfd 100755
--- a/vndk/tools/definition-tool/vndk_definition_tool.py
+++ b/vndk/tools/definition-tool/vndk_definition_tool.py
@@ -3339,19 +3339,19 @@
path_filter = re.compile(args.path_filter) if args.path_filter else None
+ if args.symbols:
+ def collect_symbols(user, definer):
+ return user.get_dep_linked_symbols(definer)
+ else:
+ def collect_symbols(user, definer):
+ return ()
+
results = []
for partition in range(NUM_PARTITIONS):
for name, lib in graph.lib_pt[partition].items():
if path_filter and not path_filter.match(name):
continue
- if args.symbols:
- def collect_symbols(user, definer):
- return user.get_dep_linked_symbols(definer)
- else:
- def collect_symbols(user, definer):
- return ()
-
data = []
if args.revert:
for assoc_lib in sorted(lib.users_all):