summaryrefslogtreecommitdiff
path: root/indra
diff options
context:
space:
mode:
authorBrad Payne (Vir Linden) <vir@lindenlab.com>2017-11-16 20:23:39 +0000
committerBrad Payne (Vir Linden) <vir@lindenlab.com>2017-11-16 20:23:39 +0000
commit6e75ceeff3b145c159f23e3a24576d0622ee4e1d (patch)
tree793370cd8fef15e2afe1f824e997ed6b8d13dffd /indra
parent64e5cfc05452993bf15a31d9d4a0deeb1414edf0 (diff)
SL-704 - more cleanup of AXON commented code
Diffstat (limited to 'indra')
-rw-r--r--indra/newview/llvovolume.cpp81
-rw-r--r--indra/newview/llvovolume.h1
2 files changed, 10 insertions, 72 deletions
diff --git a/indra/newview/llvovolume.cpp b/indra/newview/llvovolume.cpp
index 96f2255265..ee3ecb2237 100644
--- a/indra/newview/llvovolume.cpp
+++ b/indra/newview/llvovolume.cpp
@@ -3433,47 +3433,6 @@ bool LLVOVolume::isAnimatedObject() const
#endif
}
-// Make sure animated objects in a linkset are consistent. The rules are:
-// Only the root of a linkset can have the animated object flag set
-// Only the root of a linkset can have a control avatar (iff the animated object flag is set)
-// Only skinned mesh volumes can have the animated object flag set, or a control avatar
-//
-// AXON REVIEW BASED ON FINAL RULES
-bool LLVOVolume::isAnimatedObjectStateConsistent() const
-{
- if (!canBeAnimatedObject())
- {
- if ((getExtendedMeshFlags() & LLExtendedMeshParams::ANIMATED_MESH_ENABLED_FLAG) ||
- mControlAvatar.notNull())
- {
- LL_WARNS("AXON") << "Non animatable object has mesh enabled flag or mControlAvatar. Flags "
- << getExtendedMeshFlags() << " cav " << mControlAvatar.get() << LL_ENDL;
- return false;
- }
- }
- if (!isRootEdit())
- {
- if ((getExtendedMeshFlags() & LLExtendedMeshParams::ANIMATED_MESH_ENABLED_FLAG) ||
- mControlAvatar.notNull())
- {
- LL_WARNS("AXON") << "Non root object has mesh enabled flag or mControlAvatar. Flags "
- << getExtendedMeshFlags() << " cav " << mControlAvatar.get() << LL_ENDL;
- return false;
- }
- }
- // If we get here, we have a potentially animatable root volume.
- bool is_animation_enabled = getExtendedMeshFlags() & LLExtendedMeshParams::ANIMATED_MESH_ENABLED_FLAG;
- bool has_control_avatar = (mControlAvatar.notNull());
- if (is_animation_enabled != has_control_avatar)
- {
- LL_WARNS("AXON") << "Inconsistent state: animation enabled " << is_animation_enabled
- << " has control avatar " << has_control_avatar
- << " flags " << getExtendedMeshFlags() << " cav " << mControlAvatar.get() << LL_ENDL;
- return false;
- }
- return true;
-}
-
// Called any time parenting changes for a volume. Update flags and
// control av accordingly. This is called after parent has been
// changed to new_parent.
@@ -3507,25 +3466,6 @@ void LLVOVolume::updateAnimatedObjectStateOnReparent(LLViewerObject *old_parent,
old_volp->getControlAvatar()->removeAttachmentOverridesForObject(this);
}
}
-
- if (old_volp)
- {
- if (!old_volp->isAnimatedObjectStateConsistent())
- {
- LL_WARNS("AXON") << "old_volp failed consistency check" << LL_ENDL;
- }
- }
- if (new_volp)
- {
- if (!new_volp->isAnimatedObjectStateConsistent())
- {
- LL_WARNS("AXON") << "new_volp failed consistency check" << LL_ENDL;
- }
- }
- if (!isAnimatedObjectStateConsistent())
- {
- LL_WARNS("AXON") << "child object failed consistency check" << LL_ENDL;
- }
}
//----------------------------------------------------------------------------
@@ -3984,17 +3924,19 @@ void LLVOVolume::parameterChanged(U16 param_type, LLNetworkData* data, BOOL in_u
}
if (!local_origin && param_type == LLNetworkData::PARAMS_EXTENDED_MESH)
{
- // AXON better if we could compare the before and after flags directly.
U32 extended_mesh_flags = getExtendedMeshFlags();
bool enabled = (extended_mesh_flags & LLExtendedMeshParams::ANIMATED_MESH_ENABLED_FLAG);
+ // AXON This is kind of a guess. Better if we could compare
+ // the before and after flags directly. What about cases where
+ // there's no control avatar for optimization reasons?
bool was_enabled = (getControlAvatar() != NULL);
if (enabled != was_enabled)
{
LL_DEBUGS("AXON") << (U32) this
- << " calling onSetExtendedMeshFlags, enabled " << (U32) enabled
- << " was_enabled " << (U32) was_enabled
- << " local_origin " << (U32) local_origin
- << LL_ENDL;
+ << " calling onSetExtendedMeshFlags, enabled " << (U32) enabled
+ << " was_enabled " << (U32) was_enabled
+ << " local_origin " << (U32) local_origin
+ << LL_ENDL;
onSetExtendedMeshFlags(extended_mesh_flags);
}
}
@@ -5159,13 +5101,10 @@ void LLVolumeGeometryManager::rebuildGeom(LLSpatialGroup* group)
bool bake_sunlight = LLPipeline::sBakeSunlight && drawablep->isStatic();
- // AXON why this variable? Only different from rigged if
- // there are no LLFaces associated with the drawable.
- bool is_rigged = false;
+ bool any_rigged_face = false;
if (rigged && rigged_av)
{
- // AXON don't we want to do this for standalone animesh as well?
rigged_av->addAttachmentOverridesForObject(vobj);
if (!LLApp::isExiting() && rigged_av->isSelf() && debugLoggingEnabled("AvatarAttachments"))
{
@@ -5198,7 +5137,7 @@ void LLVolumeGeometryManager::rebuildGeom(LLSpatialGroup* group)
}
facep->setState(LLFace::RIGGED);
- is_rigged = true;
+ any_rigged_face = true;
//get drawpool of avatar with rigged face
LLDrawPoolAvatar* pool = get_avatar_drawpool(vobj);
@@ -5542,7 +5481,7 @@ void LLVolumeGeometryManager::rebuildGeom(LLSpatialGroup* group)
}
}
- if (is_rigged)
+ if (any_rigged_face)
{
if (!drawablep->isState(LLDrawable::RIGGED))
{
diff --git a/indra/newview/llvovolume.h b/indra/newview/llvovolume.h
index afeb1c852b..afd6c234ee 100644
--- a/indra/newview/llvovolume.h
+++ b/indra/newview/llvovolume.h
@@ -278,7 +278,6 @@ public:
void setExtendedMeshFlags(U32 flags);
bool canBeAnimatedObject() const;
bool isAnimatedObject() const;
- bool isAnimatedObjectStateConsistent() const;
void updateAnimatedObjectStateOnReparent(LLViewerObject *old_parent, LLViewerObject *new_parent);
std::map<LLUUID, S32> mObjectSignaledAnimations; // requested state of Animation name/value