Closed Bug 418981 Opened 13 years ago Closed 13 years ago

CompressWhitespace should be defined for nsACString too.

Categories

(Core :: XPCOM, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: prasad, Assigned: prasad)

References

Details

Attachments

(1 file)

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.
The new function CompressWhitespace(nsACString) is just a clone of CompressWhitespace(nsAString) that already exists.
Attachment #304913 - Flags: review?(benjamin)
Status: NEW → ASSIGNED
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: 13 years ago
Resolution: --- → WONTFIX
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.