Closed
Bug 941598
Opened 6 years ago
Closed 6 years ago
constify PLDHashTableOps in netwerk/
Categories
(Core :: Networking, defect)
Core
Networking
Not set
Tracking
()
RESOLVED
FIXED
mozilla28
People
(Reporter: froydnj, Assigned: froydnj)
Details
(Whiteboard: [qa-])
Attachments
(1 file)
|
6.07 KB,
patch
|
jduell.mcbugs
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
||
Updated•6 years ago
|
Attachment #8336017 -
Flags: review?(jduell.mcbugs) → review+
|
Assignee | |
Comment 2•6 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/6f5bf94c608c
Flags: in-testsuite-
|
|
Assignee | |
Updated•6 years ago
|
Assignee: nobody → nfroyd
|
||
Comment 3•6 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/6f5bf94c608c
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
|
|
||
Comment 4•6 years ago
|
||
If I was doing this work, I'd definitely do all the patches in one bug, and probably even put them all into one patch. I understand that you're trying to get module peers to do the reviews, but for a change this simple and mechanical I think it's perfectly reasonable to ask someone experienced to review the changes across all modules in one hit.
|
||
Updated•6 years ago
|
Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•