Closed Bug 418981 Opened 13 years ago Closed 13 years ago
Whitespace should be defined for ns ACString too .
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)
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.