----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/492/ -----------------------------------------------------------
(Updated Oct. 12, 2011, 5:19 p.m.) Review request for Viewer, Oz Linden, Nyx Linden, Jenn, Richard Nelson, and Brad Payne. Changes ------- Fixing parent diff. 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 (updated) ----- 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 ------- 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