-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/545/
-----------------------------------------------------------
(Updated Feb. 3, 2012, 8:51 a.m.)
Review request for Viewer.
Changes
-------
Fixed another logic error in getAvatars. This code could be improved by
changing one call into this method, so avatar_ids always has a matching
positions parameter (which, erroneously, I thought it did).
Description
-------
The SL simulator has recently been fixed so that the CoarseLocationUpdate
message properly returns 255 for all altitudes above 1020 meters.
The code for the mini-map, in drawing the agent locations for equal, above or
below needs to take this into account. It currently does not.
The routine that returns who is nearby can be enhanced to scan the character
list and use that position data. This gives an accurate z-level value, even
when above 1020m.
In the case where the relative Z value between you and another avatar is
unknown display an X on the mini-map.
This addresses bug STORM-1793.
http://jira.secondlife.com/browse/STORM-1793
Diffs (updated)
-----
doc/contributions.txt 0010858de5a1
indra/newview/llnetmap.cpp 0010858de5a1
indra/newview/llworld.cpp 0010858de5a1
indra/newview/llworldmapview.h 0010858de5a1
indra/newview/llworldmapview.cpp 0010858de5a1
Diff: http://codereview.secondlife.com/r/545/diff/diff
Testing
-------
See test plan in jira
Thanks,
Jonathan Yap
_______________________________________________
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
