Closed
Bug 373496
Opened 18 years ago
Closed 18 years ago
"dead" emoticon encoding in mozTXTToHTMLConv.cpp
Categories
(MailNews Core :: Backend, defect)
MailNews Core
Backend
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bugzilla, Assigned: bugzilla)
Details
Attachments
(1 file)
|
1.04 KB,
patch
|
Bienvenu
:
review+
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.2) Gecko/20070222 SeaMonkey/1.1.1 Mnenhy/0.7.5.0
Build Identifier:
In mozTXTToHTMLConv.cpp's line 889, the emoticon ">:-o" is interpreted as "moz-smiley-yell", which does not exist and therefore doesn't work. This should obviously be "moz-smiley-s10", see context.
Reproducible: Always
|
||
Comment 1•18 years ago
|
||
> In mozTXTToHTMLConv.cpp's line 889, the emoticon ">:-o" is interpreted as
> "moz-smiley-yell", which does not exist and therefore doesn't work. This should
> obviously be "moz-smiley-s10", see context.
<http://lxr.mozilla.org/seamonkey/source/netwerk/streamconv/converters/mozTXTToHTMLConv.cpp#882>
Assignee: nobody → bugzilla
Status: UNCONFIRMED → NEW
Ever confirmed: true
|
Assignee | |
Comment 2•18 years ago
|
||
Attachment #258156 -
Flags: superreview?(bienvenu)
Attachment #258156 -
Flags: review?(bienvenu)
|
||
Comment 3•18 years ago
|
||
Comment on attachment 258156 [details] [diff] [review]
replaces "moz-smiley-yell" with "moz-smiley-s10" for ">:-o"
moz-smiley-yell doesn't seem to exist, you're right.
Attachment #258156 -
Flags: superreview?(bienvenu)
Attachment #258156 -
Flags: superreview+
Attachment #258156 -
Flags: review?(bienvenu)
Attachment #258156 -
Flags: review+
|
|
Assignee | |
Updated•18 years ago
|
Whiteboard: [checkin needed]
|
|
||
Comment 4•18 years ago
|
||
Landed on trunk.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed]
|
||
Updated•17 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•