Closed
Bug 174219
Opened 23 years ago
Closed 22 years ago
minor cleanup for nsSVGLength::SetValueAsString(const nsAString & aValueAsString)
Categories
(Core :: SVG, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: timeless, Assigned: alex)
Details
301 // XXX how am I supposed to do this ???
302 // char* str = aValue.ToNewCString();
answer:
str = ToNewCString(aValue)
303 char* str;
304 {
305 nsAutoString temp(aValueAsString);
306 str = ToNewCString(temp);
307 }
308
309 char* number = str;
310 while (*number && isspace(*number))
This crashes in the OOM case. (And for bonus points if it didn't, it would crash
on os/2 at the end.)
335 nsMemory::Free(str);
Comment 1•23 years ago
|
||
I don't get the OS2 comment
| Assignee | ||
Comment 2•22 years ago
|
||
This has been fixed for a while now.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•