Closed Bug 1480323 Opened Last year Closed Last year

Reorder methods in Hash{Set,Map}

Categories

(Core :: MFBT, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: njn, Assigned: njn)

References

Details

Attachments

(1 file)

Despite extensive experience with Hash{Set,Map}, I only just realized the existence of various methods, such as has() and put(). I think this is because they are low down in the class definitions, below functions like lookup() and add().

The classes will be easier to understand if the methods are rearranged and put into more logical groups.
Comment on attachment 8997232 [details]
Bug 1480323 - Reorder methods in Hash{Set,Map}.

https://reviewboard.mozilla.org/r/261110/#review268264
Attachment #8997232 - Flags: review?(luke) → review+
https://hg.mozilla.org/mozilla-central/rev/3abea9c60292
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.