[opensource-dev] Review Request: VWR-26393 /me emote doubles name in chat history

[Vadim ProductEngine]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/419/
-----------------------------------------------------------

(Updated Aug. 15, 2011, 7:19 a.m.)


Review request for Viewer.


Summary (updated)
-------

As a resident of Second Life I find the displaying an emoter's name twice in nearby chat history rather unnecessary.  Please lay eyes on this proposed code change and let me know what you think.

I took care to remove the unlinked sender name instead of the linked one.


This addresses bug VWR-26393.
    http://jira.secondlife.com/browse/VWR-26393


Diffs
-----

  indra/newview/llchathistory.cpp 2f9a87538ca1 

Diff: http://codereview.secondlife.com/r/419/diff


Testing
-------

Tell a crowd what you are testing and many will start talking to you prefixed with /me, if they are good sports.  I have been using this change for 3 days now and have not noticed any ill effects.  Please note: This changest is not intended to affect nearby chat toasts, nearby chat log files, IM history, IM toasts or IM log files.


Thanks,

ardy.lay

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110815/5f8590ac/attachment-0001.htm