Closed Bug 1128407 (modernize-pldhash) Opened 5 years ago Closed 4 years ago

Modernize PLDHashTable

Categories

(Core :: XPCOM, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: njn, Assigned: njn)

References

Details

PLDHashTable was originally written in old-fashioned C. It's now C++ but still has a lot of C-isms. There's been a bunch of work recently to remove some of these (and make it nicer to use in other ways) and I have some more planned. Might as well track them in one place.
Depends on: 1050035
Depends on: 1131199
Some other things that could be done:
- replace some of these macro constants with actual constants
- INCREMENT_RECURSION_LEVEL and DECREMENT_RECURSION_LEVEL should be called in some RAII class.
Depends on: 1131300
Depends on: 1131302
Depends on: 1131308
Depends on: 1131901
Depends on: 1159972
Depends on: 1160436
Depends on: 1161377
Depends on: 1165768
Depends on: 1165786
Alias: modernize-pldhash
Depends on: 1121760
Depends on: 1166586
Depends on: 1166598
Depends on: 1168007
Depends on: 1170416
Depends on: 1172761
Depends on: 1171833
Depends on: 1171832
Depends on: 1171830
Depends on: 1172784
Depends on: 1172789
Depends on: 1173212
Depends on: 1173247
Depends on: 1173600
Depends on: 1174594
Depends on: 1174625
Depends on: 1175410
Depends on: 1175771
Depends on: 1175807
Depends on: 1175810
Depends on: 1176159
Depends on: 1176160
Depends on: 1179071
Depends on: 1179657
Depends on: 1180071
Depends on: 1180072
Depends on: 1180084
Depends on: 1181411
Depends on: 1185399
Depends on: 1187197
Depends on: 1187767
Depends on: 1189156
Depends on: 1201135
Depends on: 1202526
All blocking bugs have been fixed. I think we're done here.
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.