Closed Bug 1172789 Opened 10 years ago Closed 10 years ago

Remove PL_DHashTableEnumerate() uses from nsLoadGroup

Categories

(Core :: Networking, defect)

Product:
Core
Component:
Networking
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla41
Tracking Flags:
Tracking Status
firefox41 --- fixed

People

(Reporter: n.nethercote, Assigned: n.nethercote)

References

Details

Attachments

(3 files, 3 obsolete files)

(part 2) - Remove PL_DHashTableEnumerate() uses from nsLoadGroup
2.00 KB, patch
michal
: review+
Details | Diff | Splinter Review
(part 3) - Remove PL_DHashTableEnumerate() uses from nsLoadGroup
4.50 KB, patch
michal
: review+
Details | Diff | Splinter Review
(part 1) - Remove PL_DHashTableEnumerate() uses from nsLoadGroup
1.69 KB, patch
michal
: review+
Details | Diff | Splinter Review
Because PLDHashTable::Iterate is nicer than PL_DHashTableEnumerate().
Attachment #8617118 - Flags: review?(cbiesinger)
This change also factors out some duplicated code.
Attachment #8617120 - Flags: review?(michal.novotny)
Attachment #8617118 - Flags: review?(cbiesinger)
Attachment #8617118 - Flags: review?(michal.novotny)
Attachment #8617119 - Attachment is obsolete: true
Attachment #8617119 - Flags: review?(michal.novotny)
This change also factors out some duplicated code.
Attachment #8617161 - Flags: review?(michal.novotny)
Attachment #8617120 - Attachment is obsolete: true
Attachment #8617120 - Flags: review?(michal.novotny)
Attachment #8617118 - Attachment is obsolete: true
Attachment #8617118 - Flags: review?(michal.novotny)
Attachment #8617160 - Flags: review?(michal.novotny) → review+
Attachment #8617163 - Flags: review?(michal.novotny) → review+
Attachment #8617161 - Flags: review?(michal.novotny) → review+
Depends on: 1174046
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: