-----------------------------------------------------------
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
_______________________________________________
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges
