Core: fix dangerous mutable default in Fill (#5247)

discussed here https://discord.com/channels/731205301247803413/731214280439103580/1327712564213448775
This commit is contained in:
Doug Hoskisson
2025-07-28 06:41:43 -07:00
committed by GitHub
parent 5e2702090c
commit 4d17366662

View File

@@ -358,7 +358,12 @@ def fast_fill(multiworld: MultiWorld,
return item_pool[placing:], fill_locations[placing:] return item_pool[placing:], fill_locations[placing:]
def accessibility_corrections(multiworld: MultiWorld, state: CollectionState, locations, pool=[]): def accessibility_corrections(multiworld: MultiWorld,
state: CollectionState,
locations: list[Location],
pool: list[Item] | None = None) -> None:
if pool is None:
pool = []
maximum_exploration_state = sweep_from_pool(state, pool) maximum_exploration_state = sweep_from_pool(state, pool)
minimal_players = {player for player in multiworld.player_ids if minimal_players = {player for player in multiworld.player_ids if
multiworld.worlds[player].options.accessibility == "minimal"} multiworld.worlds[player].options.accessibility == "minimal"}