If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

CompressWhitespace should be defined for nsACString too.

RESOLVED WONTFIX

Status

()

Core
XPCOM
RESOLVED WONTFIX
10 years ago
10 years ago

People

(Reporter: prasad, Assigned: prasad)

Tracking

(Blocks: 1 bug)

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 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

10 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

10 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 2

10 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: 10 years ago
Resolution: --- → WONTFIX

Comment 3

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