Chat log viewer wrongly parses non-emoticon text as emoticons

[pi++]

I've noticed that with svn-revision 23835, when opening the Chat log viewer:
1) The first timestamp, if it contains text that is an emoticon equivalent (things like :1, :0, :2), is shown with an emoticon inside it. That seems to happen only for the minutes (hh:Mm:ss), never for the seconds.

2) Sometimes emoticons are displayed in timestamps following status messages like "State changed to 'offline' (hh:mm:ss)". It happens only sometimes, not always... I can't understand the pattern behind it.

3) What I think is a default ICQ emoticon ( O:-) ) is rendered as a sequence of two emoticons ( :O :-) ) when placed like this ( (hh:mm:ss) NickName:O:-) )

See you,
Pietro

[pi++]

Case 1 - First timestamp

[pi++]

Case 2 - State change message

[pi++]

Case 3a - ICQ Emoticon

[pi++]

Case 3b - ICQ Emoticon (as text)

[Robby]

Shouldn't this be considered a regression?

[zacw]

(In [23848]) Making the Default emoticon set a bit more conservative. Refs #9989.

[zacw]

(In [23849]) ':l' too. Refs #9989.

[evands]

(In [23850]) Decode messages separately, and filter them on their own, rather than decoding and filtering the entire displayed chat log. This lets us not emoticonify and otherwise modify timestamps, sender names, and statuses. Fixes #9989

[anonymous]

Milestone SVN issues deleted