summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rwxr-xr-xindra/newview/llattachmentsmgr.cpp9
-rwxr-xr-xindra/newview/llattachmentsmgr.h5
-rw-r--r--indra/newview/llavatarrendernotifier.cpp4
-rwxr-xr-xindra/newview/llvoavatar.cpp2
4 files changed, 12 insertions, 8 deletions
diff --git a/indra/newview/llattachmentsmgr.cpp b/indra/newview/llattachmentsmgr.cpp
index 2a137cc39b..d3e66289d1 100755
--- a/indra/newview/llattachmentsmgr.cpp
+++ b/indra/newview/llattachmentsmgr.cpp
@@ -421,6 +421,15 @@ void LLAttachmentsMgr::onDetachCompleted(const LLUUID& inv_item_id)
mQuestionableCOFLinks.addTime(inv_item_id);
}
+bool LLAttachmentsMgr::isAttachmentStateComplete() const
+{
+ return mPendingAttachments.empty()
+ && mAttachmentRequests.empty()
+ && mDetachRequests.empty()
+ && mRecentlyArrivedAttachments.empty()
+ && mQuestionableCOFLinks.empty();
+}
+
// Check for attachments that are (a) linked in COF and (b) not
// attached to the avatar. This is a rotten function to have to
// include, because it runs the risk of either repeatedly spamming out
diff --git a/indra/newview/llattachmentsmgr.h b/indra/newview/llattachmentsmgr.h
index fab146cb52..bb7d35edbc 100755
--- a/indra/newview/llattachmentsmgr.h
+++ b/indra/newview/llattachmentsmgr.h
@@ -87,10 +87,7 @@ public:
void onDetachRequested(const LLUUID& inv_item_id);
void onDetachCompleted(const LLUUID& inv_item_id);
- bool hasPendingAttachments() { return mPendingAttachments.size() > 0; }
- bool hasAttachmentRequests() { return mAttachmentRequests.size() > 0; }
- bool hasDetachRequests() { return mAttachmentRequests.size() > 0; }
- bool hasRecentlyArrivedAttachments() { return mRecentlyArrivedAttachments.size() > 0; }
+ bool isAttachmentStateComplete() const;
private:
diff --git a/indra/newview/llavatarrendernotifier.cpp b/indra/newview/llavatarrendernotifier.cpp
index ca3c1a7310..53be573461 100644
--- a/indra/newview/llavatarrendernotifier.cpp
+++ b/indra/newview/llavatarrendernotifier.cpp
@@ -191,9 +191,7 @@ void LLAvatarRenderNotifier::updateNotificationState()
if (mInitialCofVersion < 0
&& gAgentWearables.areWearablesLoaded()
- && !LLAttachmentsMgr::getInstance()->hasPendingAttachments()
- && !LLAttachmentsMgr::getInstance()->hasAttachmentRequests()
- && !LLAttachmentsMgr::getInstance()->hasRecentlyArrivedAttachments())
+ && LLAttachmentsMgr::getInstance()->isAttachmentStateComplete())
{
// cof formed
mInitialCofVersion = LLAppearanceMgr::instance().getCOFVersion();
diff --git a/indra/newview/llvoavatar.cpp b/indra/newview/llvoavatar.cpp
index 3e20fbecdb..da02b96f80 100755
--- a/indra/newview/llvoavatar.cpp
+++ b/indra/newview/llvoavatar.cpp
@@ -6458,7 +6458,7 @@ BOOL LLVOAvatar::processFullyLoadedChange(bool loading)
mFullyLoadedInitialized = TRUE;
mFullyLoadedFrameCounter++;
- if (isSelf())
+ if (changed && isSelf())
{
// to know about outfit switching
LLAvatarRenderNotifier::getInstance()->updateNotificationState();