Closed
Bug 455742
Opened 16 years ago
Closed 16 years ago
outparamdel nsIForm
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: taras.mozilla, Assigned: taras.mozilla)
References
Details
Attachments
(1 file)
(deleted),
patch
|
jst
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
From what I can see only ResolveName,GetElementCount,IndexOfControl control could be modified. The rest use nsresult to pass additional info. I'm also not sure what the point of DoResolveName is. I left it in for now.
Attachment #339087 -
Flags: review?(jst)
Assignee | ||
Comment 2•16 years ago
|
||
(In reply to comment #1) > + PRUint32 count = nsnull; > > Should use a plain zero. (two places) thanks, i'll fix those before commit
Updated•16 years ago
|
QA Contact: content
Updated•16 years ago
|
Attachment #339087 -
Flags: superreview+
Attachment #339087 -
Flags: review?(jst)
Attachment #339087 -
Flags: review+
Comment 3•16 years ago
|
||
Comment on attachment 339087 [details] [diff] [review] outparamdelling Looks good!
Assignee | ||
Comment 4•16 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/51c533bbac70
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Assignee: nobody → tglek
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•