Closed
Bug 157970
Opened 22 years ago
Closed 22 years ago
cleanup editor.js (and other files?) usage of "new Object"
Categories
(SeaMonkey :: Composer, defect)
SeaMonkey
Composer
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.2alpha
People
(Reporter: Brade, Assigned: Brade)
Details
Attachments
(1 file, 1 obsolete file)
|
17.02 KB,
patch
|
neil
:
review+
alecf
:
superreview+
|
Details | Diff | Splinter Review |
we should cleanup the usage of "new Object" in editor.js (and other places?)
we should use this format instead:
var foo = { value: null };
|
||
Updated•22 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.2alpha
|
|
||
Comment 1•22 years ago
|
||
Yes, we should do this. It causes JS warnings in some places. Easy task.
Keywords: nsbeta1
|
|
||
Comment 2•22 years ago
|
||
Reassigning to brade
|
Assignee | |
Comment 3•22 years ago
|
||
|
|
Assignee | |
Comment 4•22 years ago
|
||
Attachment #95574 -
Attachment is obsolete: true
|
||
Comment 5•22 years ago
|
||
Comment on attachment 95583 [details] [diff] [review] patch #2 (added initialization) sr=alecf
Attachment #95583 -
Flags: superreview+
|
||
Comment 6•22 years ago
|
||
Comment on attachment 95583 [details] [diff] [review] patch #2 (added initialization) r=neil@parkwaycc.co.uk
Attachment #95583 -
Flags: review+
|
|
Assignee | |
Comment 7•22 years ago
|
||
fix checked in
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
|
||
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•