[opensource-dev] Review Request: (STORM-380) There is a little delay in sound when gesture first time played

Oz Linden oz at lindenlab.com
Mon Mar 28 04:06:41 PDT 2011


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



indra/newview/llgesturemgr.cpp
<http://codereview.secondlife.com/r/231/#comment381>

    I think that there's a race condition here.  
    
    Wouldn't it be better to insert the id into the mLoadingAssets before requesting it, so that it is sure to be in the list if the onAssetLoadComplete is called immediately?
    



indra/newview/llgesturemgr.cpp
<http://codereview.secondlife.com/r/231/#comment382>

    Same possible race as above


- Oz


On March 25, 2011, 5:33 p.m., Seth ProductEngine wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/231/
> -----------------------------------------------------------
> 
> (Updated March 25, 2011, 5:33 p.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Summary
> -------
> 
> First pass implementation of syncing the animations and sounds before the gesture starts playing.
> The actual playing of animations and sounds of a gesture starts only when all needed animations and sound files are loaded into viewer cache. This reduces the delay between animations and sounds meant to be played simultaneously but may increase the delay between the moment a gesture is triggered and the moment it starts playing.
> 
> 
> This addresses bug STORM-380.
>     http://jira.secondlife.com/browse/STORM-380
> 
> 
> Diffs
> -----
> 
>   indra/newview/llgesturemgr.h 6c15f820c3b9 
>   indra/newview/llgesturemgr.cpp 6c15f820c3b9 
> 
> Diff: http://codereview.secondlife.com/r/231/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Seth
> 
>

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


More information about the opensource-dev mailing list