Closed
Bug 8513
Opened 26 years ago
Closed 26 years ago
[regression] related links busted throws JS exception
Categories
(SeaMonkey :: Sidebar, defect, P3)
SeaMonkey
Sidebar
Tracking
(Not tracked)
VERIFIED
FIXED
M7
People
(Reporter: waterson, Assigned: waterson)
Details
(Whiteboard: fix in hand. let me check in chofmann.)
Attachments
(1 file)
937 bytes,
text/plain
|
Details |
I debugged this; it is in fact due to Necko landing and incorrect use of
nsAutoString.
Jud: you replaced a call to nsAutoString::ToNewCString() with GetBuffer(). This
fails because it's a two-byte string, and it has no one-byte buffer. So the
string that it returns is always null. I see why you did this (to avoid cleanup
code inside the #ifdef).
Could you please re-examine this patch and fix?
Assignee | ||
Updated•26 years ago
|
Target Milestone: M7
Assignee | ||
Comment 1•26 years ago
|
||
Marked M7. This is a must-have feature in M7.
Assignee | ||
Comment 2•26 years ago
|
||
Assignee | ||
Updated•26 years ago
|
Assignee: valeski → waterson
Assignee | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Whiteboard: fix in hand. let me check in chofmann.
Assignee | ||
Comment 3•26 years ago
|
||
Allright, I have a fix. Somebody please give me permission to check in.
Comment 4•26 years ago
|
||
ok.
Assignee | ||
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•26 years ago
|
||
a=chofmann
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Comment 6•26 years ago
|
||
related links working fine in 6/19 sweetlou builds, so marking verified
Updated•21 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•