CompressWhitespace should be defined for nsACString too.

RESOLVED WONTFIX

Status

()

RESOLVED WONTFIX
11 years ago
11 years ago

People

(Reporter: prasad, Assigned: prasad)

Tracking

(Blocks: 1 bug)

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

11 years ago
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

11 years ago
Created attachment 304913 [details] [diff] [review]
Adds CompressWhitespace(nsACString)

The new function CompressWhitespace(nsACString) is just a clone of CompressWhitespace(nsAString) that already exists.
Attachment #304913 - Flags: review?(benjamin)
(Assignee)

Updated

11 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 2

11 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
Last Resolved: 11 years ago
Resolution: --- → WONTFIX

Comment 3

11 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.