Closed
Bug 418981
Opened 17 years ago
Closed 17 years ago
CompressWhitespace should be defined for nsACString too.
Categories
(Core :: XPCOM, defect)
Core
XPCOM
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: prasad, Assigned: prasad)
References
Details
Attachments
(1 file)
|
3.74 KB,
patch
|
benjamin
:
review-
|
Details | Diff | Splinter Review |
Currently a function to compress white spaces inside the string exists for nsAString and a similar function is needed for nsACString too.
xpcom/tests/external/TestMinStringAPI.cpp has a place holder which only trims the string and does not actually compress.
|
Assignee | |
Comment 1•17 years ago
|
||
The new function CompressWhitespace(nsACString) is just a clone of CompressWhitespace(nsAString) that already exists.
Attachment #304913 -
Flags: review?(benjamin)
|
|
Assignee | |
Updated•17 years ago
|
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•17 years ago
|
||
Resolution => WONTFIX
as discussed with Benjamin on the IRC, there are no other users for this function and hence it should probably be somewhere inside the mailnews code itself.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → WONTFIX
|
||
Comment 3•17 years ago
|
||
Comment on attachment 304913 [details] [diff] [review]
Adds CompressWhitespace(nsACString)
Talking on IRC, I don't like this function and would prefer that it end up in mailnews-specific code.
Attachment #304913 -
Flags: review?(benjamin) → review-
You need to log in
before you can comment on or make changes to this bug.
Description
•