454129 - Use .trim() in Editor/Composer

Status: RESOLVED FIXED

(SeaMonkey :: Composer, defect)

Description

[Mike Kaplinskiy [:mikekap] (at school, no time)]

Attachment: Patch v1

Fallout of bug 220348.

I actually have no idea if this patch <applies>. I don't have a c-c repo, so if someone wants to take this up - go ahead. attachment 332666 [details] [diff] [review] has a patch that *did* apply, but before the move of editor.

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Bv1) s/replace/trim/ only [Checkin: Comment 3]

This is untested, but should be trivial enough.

Comment 2

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Comment on attachment 355103 [details] [diff] [review]
(Bv1) s/replace/trim/ only
[Checkin: Comment 3]

r=me (trivial changes)

Comment 3

[Serge Gautherie (:sgautherie)]

Comment on attachment 355103 [details] [diff] [review]
(Bv1) s/replace/trim/ only
[Checkin: Comment 3]

http://hg.mozilla.org/comm-central/rev/fd2b49b65f9f

Comment 4

[Serge Gautherie (:sgautherie)]

(In reply to comment #0)
> Patch v1

Do we really want to inline these 3 functions ?
If yes, we may prefer to use "if (s)  s.trim();" when there is a var !?

Comment 5

[Frank-Rainer Grahl (:frg)]

The regex is gone and the remaining fucntions can stay I think. So call it closed.