Bug 1161377 - PLDHashTable: Add and use an initializing constructor and destructor https://hg.mozilla.org/mozilla-central/rev/2eae1608bcfa
Bug 1161377 removed PL_NewDHashTable() and PL_DHashTableDestroy() Replace with multi-arg constructor
Attachment #8603754 - Flags: review?(rkent)
Sorry about this. I was going to write a patch on Friday for c-c but a fresh checkout didn't compile and then I got distracted and then Friday was over and it was the weekend.
Comment on attachment 8603754 [details] [diff] [review] Patch v1.0 Proposed fix [Checked in: Comment 4] I did not try to compile this with the other ongoing breakage, but the changes are pretty straightforward. r=me
Attachment #8603754 - Flags: review?(rkent) → review+
OS: Unspecified → All
Hardware: Unspecified → All
Comment on attachment 8603754 [details] [diff] [review] Patch v1.0 Proposed fix [Checked in: Comment 4] https://hg.mozilla.org/comm-central/rev/95616afdd719
Attachment #8603754 - Attachment description: Patch v1.0 Proposed fix → Patch v1.0 Proposed fix [Checked in: Comment 4]
Bug 1161377's patches were backed out, so this will have to backed out at some point too. Sorry :(
Backed out https://hg.mozilla.org/comm-central/rev/fcbd8eb059be due to backout of m-c bug. Reopening since we'll probably want to reland later.
The new PLDHashTable API changes are in bug 1170416.
Assignee: philip.chee → n.nethercote
Status: REOPENED → ASSIGNED
Attachment #8603754 - Attachment is obsolete: true
Comment on attachment 8614513 [details] [diff] [review] Update /mailnews/ for PLDHashTable API changes https://hg.mozilla.org/comm-central/rev/b3a333547c76 I pushed this since we are busted without it.
Attachment #8614513 - Flags: review?(rkent) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago → 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.