[opensource-dev] Review Request: EXP-625 Speed up initial avatar copy and bake by moving the initial gestures copy into LLApperanceMgr::onFirstFullyVisible()

Stone Linden stone at lindenlab.com
Wed Oct 12 17:11:59 PDT 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/492/
-----------------------------------------------------------

Review request for Viewer, Oz Linden, Nyx Linden, Jenn, Richard Nelson, and Brad Payne.


Summary
-------

Move copyLibraryGestures from llstartup.cpp to llappearancemgr.cpp, and call it from the onFirstFullyVisible() method. This gives the avatar initial outfit load and bake time to finish before going into the somewhat expensive gesture copy process.


This addresses bug EXP-625.
    http://jira.secondlife.com/browse/EXP-625


Diffs
-----

  indra/newview/llagentwearables.h 88cf7d9a9d31 
  indra/newview/llagentwearables.cpp 88cf7d9a9d31 
  indra/newview/llappearancemgr.h 88cf7d9a9d31 
  indra/newview/llappearancemgr.cpp 88cf7d9a9d31 
  indra/newview/llstartup.h 88cf7d9a9d31 
  indra/newview/llstartup.cpp 88cf7d9a9d31 

Diff: http://codereview.secondlife.com/r/492/diff


Testing
-------

Fully loads and bakes my initial avatar in 20s from accepting ToS until visible in-world vs. 50s without this change. Time until Gestures are fully copies is about 45s, so slightly faster than trying to do it all at once, and a better experience to get the avatar first, then the gestures second.


Thanks,

Stone

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20111013/4a459b08/attachment.htm 


More information about the opensource-dev mailing list