SH-2689 FIX - a bit more logging and related cleanup. Somewhat arbitrarily calling this done, although tweaks will be ongoing

2 files changed, 6 insertions, 5 deletions

diff --git a/indra/newview/llvoavatar.cpp b/indra/newview/llvoavatar.cpp
index fcb774f0e5..8ea0fb7c50 100755
--- a/indra/newview/llvoavatar.cpp
+++ b/indra/newview/llvoavatar.cpp
@@ -4004,7 +4004,7 @@ void LLVOAvatar::updateVisibility()
 			LLNameValue* firstname = getNVPair("FirstName");
 			if (firstname)
 			{
-				llinfos << avString() << " updating visiblity" << llendl;
+				llinfos << avString() << " updating visibility" << llendl;
 			}
 			else
 			{
@@ -4216,7 +4216,7 @@ U32 LLVOAvatar::renderSkinned(EAvatarRenderPass pass)
 		LLNameValue* firstname = getNVPair("FirstName");
 		if (firstname)
 		{
-			llinfos << "Avatar " << firstname->getString() << " in render" << llendl;
+			llinfos << avString() << " in render" << llendl;
 		}
 		else
 		{
@@ -7231,8 +7231,9 @@ void LLVOAvatar::processAvatarAppearance( LLMessageSystem* mesgsys )
 	}
 
 
-	// runway - is this right? Will be called every time *except* the first.
-	// FIXME - trying toggle
+	// runway - was
+	// if (!is_first_appearance_message )
+	// which means it would be called on second appearance message - probably wrong.
 	if (is_first_appearance_message )
 	{
 		onFirstTEMessageReceived();
diff --git a/indra/newview/llvoavatarself.cpp b/indra/newview/llvoavatarself.cpp
index fcdb4e3c63..e0af6c3995 100755
--- a/indra/newview/llvoavatarself.cpp
+++ b/indra/newview/llvoavatarself.cpp
@@ -2339,7 +2339,7 @@ void LLVOAvatarSelf::outputRezDiagnostics() const
 void LLVOAvatarSelf::outputRezTiming(const std::string& msg) const
 {
 	llinfos
-		<< " Avatar '" << getFullname() << "' "
+		<< avString()
 		<< llformat("%s. Time from avatar creation: %.2f", msg.c_str(), mDebugSelfLoadTimer.getElapsedTimeF32())
 		<< llendl;
 }