> On Feb. 27, 2011, 5:57 a.m., Boroondas Gupte wrote: > >
Works as advertised for me in use in world and I've not found any adverse effects. Aleric's rewriting (with comment) makes the most sense. - Twisted ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/153/#review392 ----------------------------------------------------------- On Feb. 19, 2011, 9:32 a.m., ardy.lay wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/153/ > ----------------------------------------------------------- > > (Updated Feb. 19, 2011, 9:32 a.m.) > > > Review request for Viewer. > > > Summary > ------- > > https://jira.secondlife.com/browse/VWR-24917 > I have been finding the redundent display of functionally equivalent names in > nearby chat history and IM history quite tiresome. > Simple proposal: If the resident's Display Name is at the default then do not > display their user.name. > https://bitbucket.org/ArdyLay/viewer-development-vwr-24917 > > Change is to: LLAvatarName::getCompleteName > > I find the following Callers: > LLAvatarActions::requestFriendshipDialog > LLAvatarActions::startIM > LLAvatarActions::startCall > LLIMModel::LLIMSession > LLIMModel::logToFile > LLPostponedNotification::onAvatarNameCache > LLUrlEntryAgent::onAvatarNameCache > LLUrlEntryAgent::getLabel > LLUrlEntryAgentCompleteName::getName > > // Callback for name resolution of a god/estate message > llviewermessage.cpp(2149): args["NAME"] = av_name.getCompleteName(); > llviewermessage.cpp(2154): chat.mText = av_name.getCompleteName() + ": " + > message; > > static void on_avatar_name_cache_toast ... > llimview.cpp(108): args["FROM"] = av_name.getCompleteName(); > > Some of these make me wonder if this change will cause some defects and > should be implimented as a seperate function. > > > This addresses bug VWR-24917. > http://jira.secondlife.com/browse/VWR-24917 > > > Diffs > ----- > > doc/contributions.txt c10d5e37db1e > indra/llcommon/llavatarname.cpp c10d5e37db1e > > Diff: http://codereview.secondlife.com/r/153/diff > > > Testing > ------- > > I have been using this trivial change and have shared it with a friend, via > bitbucket. We have both built the viewer on Windows 7 and find the resulting > reduction in redundent text in chat and IM history on screen to be very > helpful. > > > Thanks, > > ardy.lay > >
_______________________________________________ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
