danvics/ytdl-sub
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

do not double evaluate dependent single variaibles

Browse source
This commit is contained in:
Jesse Bannon 2026-01-22 22:50:53 -08:00
parent 44705b622b
commit 356f278be9
1 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/ytdl_sub/script/script.py
View file

@ -737,15 +737,16 @@ class Script:
for variable in list(to_partially_resolve): for variable in list(to_partially_resolve):
definition = unresolved[variable] definition = unresolved[variable]
maybe_resolved = definition maybe_resolved = definition
if isinstance(definition, Variable) and definition.name not in unresolvable: if isinstance(definition, Variable) and definition.name not in unresolvable:
if definition in resolved: if definition in resolved:
maybe_resolved = resolved[definition] maybe_resolved = resolved[definition]
elif definition in unresolved:
maybe_resolved = unresolved[definition]
else: else:
raise UNREACHABLE # If it's not in resolved, it must be in unresolved.
# Do not modify the definition, this avoids duplicate work because it will
# wait for the single dependent variable to be resolved.
assert definition in unresolved
elif isinstance(definition, VariableDependency): elif isinstance(definition, VariableDependency):
maybe_resolved = definition.partial_resolve( maybe_resolved = definition.partial_resolve(
resolved_variables=resolved, resolved_variables=resolved,