-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/492/
-----------------------------------------------------------
(Updated Oct. 12, 2011, 5:24 p.m.)
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 be1660e54fa8
indra/newview/llagentwearables.cpp be1660e54fa8
indra/newview/llappearancemgr.h be1660e54fa8
indra/newview/llappearancemgr.cpp be1660e54fa8
indra/newview/llstartup.h be1660e54fa8
indra/newview/llstartup.cpp be1660e54fa8
Diff: http://codereview.secondlife.com/r/492/diff
Testing (updated)
-------
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.
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
