Closed
Bug 106787
Opened 23 years ago
Closed 10 years ago
remove nsIAllocator
Categories
(Core :: XPCOM, defect, P5)
Tracking
()
RESOLVED
FIXED
mozilla31
People
(Reporter: dougt, Assigned: Jackneill)
Details
(Whiteboard: [mentor=benjamin@smedbergs.us][lang=c++][good first bug])
Attachments
(1 file)
3.94 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
cvs remove nsIAllocator and fix all callers.
Reporter | ||
Updated•23 years ago
|
Target Milestone: --- → mozilla1.0
Reporter | ||
Updated•23 years ago
|
Keywords: mozilla1.0
Reporter | ||
Comment 1•23 years ago
|
||
not required for mozilla 1.0.
Target Milestone: mozilla1.0 → Future
Updated•18 years ago
|
QA Contact: scc → xpcom
Updated•11 years ago
|
Priority: -- → P5
Target Milestone: Future → ---
Comment 3•10 years ago
|
||
I believe there are no more uses of nsIAllocator in the tree. This file needs to be removed: http://mxr.mozilla.org/mozilla-central/source/xpcom/base/nsIAllocator.h As do the references from the following files: http://mxr.mozilla.org/mozilla-central/search?string=nsiallocato
Whiteboard: [mentor=benjamin@smedbergs.us][lang=c++][good first bug]
Assignee | ||
Comment 4•10 years ago
|
||
Hi, I'd like to work on this bug.
Comment 5•10 years ago
|
||
(In reply to Jackneill from comment #4) > Hi, I'd like to work on this bug. Great! I assigned the bug to you. Feel free to ask questions here or on IRC. Once you have something, you can attach it as a patch and set the review flag to bsmedberg.
Assignee: nobody → jackneill1000
Assignee | ||
Comment 6•10 years ago
|
||
Attachment #8406181 -
Flags: review?(benjamin)
Comment 7•10 years ago
|
||
Comment on attachment 8406181 [details] [diff] [review] bug-106787-fix.patch woot
Attachment #8406181 -
Flags: review?(benjamin) → review+
Updated•10 years ago
|
Keywords: checkin-needed
Comment 8•10 years ago
|
||
landed on mozilla-inbound https://hg.mozilla.org/integration/mozilla-inbound/rev/bb7feacef303 Mark, Congrats to your first patch and thanks (and welcome) for contributing to Mozilla!
Keywords: checkin-needed
Comment 9•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/bb7feacef303
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
You need to log in
before you can comment on or make changes to this bug.
Description
•