LLWorld::getAvatars didn't make much sense. What's with that break in the old code? Anyhow, VOAvatar position should override regionlist, probably.
This commit is contained in:
@@ -1411,6 +1411,8 @@ static LLVector3d unpackLocalToGlobalPosition(U32 compact_local, const LLVector3
|
||||
|
||||
void LLWorld::getAvatars(std::vector<LLUUID>* avatar_ids, std::vector<LLVector3d>* positions, const LLVector3d& relative_to, F32 radius) const
|
||||
{
|
||||
F32 radius_squared = radius * radius;
|
||||
|
||||
if(avatar_ids != NULL)
|
||||
{
|
||||
avatar_ids->clear();
|
||||
@@ -1419,6 +1421,33 @@ void LLWorld::getAvatars(std::vector<LLUUID>* avatar_ids, std::vector<LLVector3d
|
||||
{
|
||||
positions->clear();
|
||||
}
|
||||
// get the list of avatars from the character list first, so distances are correct
|
||||
// when agent is above 1020m and other avatars are nearby
|
||||
for (std::vector<LLCharacter*>::iterator iter = LLCharacter::sInstances.begin();
|
||||
iter != LLCharacter::sInstances.end(); ++iter)
|
||||
{
|
||||
LLVOAvatar* pVOAvatar = (LLVOAvatar*) *iter;
|
||||
if(!pVOAvatar->isDead() && !pVOAvatar->isSelf())
|
||||
{
|
||||
LLUUID uuid = pVOAvatar->getID();
|
||||
if(!uuid.isNull())
|
||||
{
|
||||
LLVector3d pos_global = pVOAvatar->getPositionGlobal();
|
||||
if(dist_vec_squared(pos_global, relative_to) <= radius_squared)
|
||||
{
|
||||
if(positions != NULL)
|
||||
{
|
||||
positions->push_back(pos_global);
|
||||
}
|
||||
if(avatar_ids !=NULL)
|
||||
{
|
||||
avatar_ids->push_back(uuid);
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
// region avatars added for situations where radius is greater than RenderFarClip
|
||||
for (LLWorld::region_list_t::const_iterator iter = LLWorld::getInstance()->getRegionList().begin();
|
||||
iter != LLWorld::getInstance()->getRegionList().end(); ++iter)
|
||||
{
|
||||
@@ -1428,50 +1457,16 @@ void LLWorld::getAvatars(std::vector<LLUUID>* avatar_ids, std::vector<LLVector3d
|
||||
for (S32 i = 0; i < count; i++)
|
||||
{
|
||||
LLVector3d pos_global = unpackLocalToGlobalPosition(regionp->mMapAvatars.get(i), origin_global);
|
||||
if(dist_vec(pos_global, relative_to) <= radius)
|
||||
if(dist_vec_squared(pos_global, relative_to) <= radius_squared)
|
||||
{
|
||||
if(positions != NULL)
|
||||
{
|
||||
positions->push_back(pos_global);
|
||||
}
|
||||
if(avatar_ids != NULL)
|
||||
{
|
||||
avatar_ids->push_back(regionp->mMapAvatarIDs.get(i));
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
// retrieve the list of close avatars from viewer objects as well
|
||||
// for when we are above 1000m, only do this when we are retrieving
|
||||
// uuid's too as there could be duplicates
|
||||
if(avatar_ids != NULL)
|
||||
{
|
||||
for (std::vector<LLCharacter*>::iterator iter = LLCharacter::sInstances.begin();
|
||||
iter != LLCharacter::sInstances.end(); ++iter)
|
||||
{
|
||||
LLVOAvatar* pVOAvatar = (LLVOAvatar*) *iter;
|
||||
if(pVOAvatar->isDead() || pVOAvatar->isSelf())
|
||||
continue;
|
||||
LLUUID uuid = pVOAvatar->getID();
|
||||
if(uuid.isNull())
|
||||
continue;
|
||||
LLVector3d pos_global = pVOAvatar->getPositionGlobal();
|
||||
if(dist_vec(pos_global, relative_to) <= radius)
|
||||
{
|
||||
bool found = false;
|
||||
uuid_vec_t::iterator sel_iter = avatar_ids->begin();
|
||||
for (; sel_iter != avatar_ids->end(); sel_iter++)
|
||||
{
|
||||
if(*sel_iter == uuid)
|
||||
{
|
||||
found = true;
|
||||
break;
|
||||
}
|
||||
}
|
||||
if(!found)
|
||||
LLUUID uuid = regionp->mMapAvatarIDs.get(i);
|
||||
// if this avatar doesn't already exist in the list, add it
|
||||
if(uuid.notNull() && avatar_ids!=NULL && std::find(avatar_ids->begin(), avatar_ids->end(), uuid) == avatar_ids->end())
|
||||
{
|
||||
if(positions != NULL)
|
||||
{
|
||||
positions->push_back(pos_global);
|
||||
}
|
||||
avatar_ids->push_back(uuid);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user