change classSymbols and instanceSymbols from lists to maps

RESOLVED INCOMPLETE

Status

defect
RESOLVED INCOMPLETE
5 years ago
3 years ago

People

(Reporter: till, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

5 years ago
The `linkSymbols` helper `link` iterates over these lists for each trait symbol, making for quadratic complexity. We can make that linear by turning those lists into maps.

Additionally, the keys of those maps could have boolean values, indicating if they are release symbols. `if (!containsSymbol(symbols, trait.name.name))` could then be replaced by a simple `if (symbols[trait.name.name] !== release)`.

We only have 10 instanceSymbol fields with non-null values, so this should be pretty quick.
Product: Firefox → Firefox Graveyard
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.