All submissions to this site are governed by the Second Life Viewer Contribution Agreement. By submitting patches and other information using this site, you acknowledge that you have read, understood, and agreed to those terms.

Review Board 1.6.11

Welcome to the Second Life Viewer Code Review tool.
See the documentation on our wiki for how to use this site.

EXP-625 Speed up initial avatar copy and bake by moving the initial gestures copy into LLApperanceMgr::onFirstFullyVisible()

Review Request #492 - Created Oct. 12, 2011 and submitted

Stone Linden Reviewers
viewer
EXP-625 oz.linden, nyx.linden, jenn, richard.linden, vir.linden
None viewer-development
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.
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 copied is about 45s.

Slightly faster than trying to do it all at once, and a better experience to get the avatar first, then the gestures second.
Review request changed
Updated (Oct. 12, 2011, 5:24 p.m.)
  • 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.

    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 copied is about 45s.
    
    Slightly faster than trying to do it all at once, and a better experience to get the avatar first, then the gestures second.