[opensource-dev] Review Request: As a music fan, I want audio to fade in gently so my immersion is increased
Jonathan Yap
jhwelch at gmail.com
Fri Dec 2 02:36:24 PST 2011
> On Dec. 1, 2011, 3:58 p.m., Callum Prentice wrote:
> > indra/newview/llvieweraudio.cpp, line 213
> > <http://codereview.secondlife.com/r/520/diff/1/?file=7434#file7434line213>
> >
> > Minor point - maybe a warning where you define the fade in/out times that they should not be zero, even for testing.
Good catch! In code that is now removed I was not letting this value go below zero. I've added a bit of code to prevent it from dropping below 0.01 as a preventative measure against future changes or TPVs who pick this up and add back the two debug settings.
- Jonathan
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/520/#review1104
-----------------------------------------------------------
On Nov. 23, 2011, 5:59 a.m., Jonathan Yap wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/520/
> -----------------------------------------------------------
>
> (Updated Nov. 23, 2011, 5:59 a.m.)
>
>
> Review request for Viewer.
>
>
> Description
> -------
>
> Audio fading in has been added for when a music stream starts. Audio fading out has been added for when there is a change in the music stream that is playing.
>
> Two dead files have been eliminated.
>
> An existing bug in how music was not being paused correctly has been fixed.
>
> When you are teleporting you will hear the music stream from the place you are leaving. This is a change in behavior; previously the music stream was stopped when the teleport progress bar was being displayed.
>
> This code change affects several areas where music is started or stopped. The new code has evolved significantly, so please look for things that might not "make sense."
>
>
> This addresses bug STORM-591.
> http://jira.secondlife.com/browse/STORM-591
>
>
> Diffs
> -----
>
> doc/contributions.txt 8b455c1b7a5e
> indra/newview/lloverlaybar.h 8b455c1b7a5e
> indra/newview/lloverlaybar.cpp 8b455c1b7a5e
> indra/newview/llpanelnearbymedia.h 8b455c1b7a5e
> indra/newview/llpanelnearbymedia.cpp 8b455c1b7a5e
> indra/newview/llvieweraudio.h 8b455c1b7a5e
> indra/newview/llvieweraudio.cpp 8b455c1b7a5e
> indra/newview/llviewermedia.cpp 8b455c1b7a5e
> indra/newview/llviewerparcelmgr.cpp 8b455c1b7a5e
>
> Diff: http://codereview.secondlife.com/r/520/diff/diff
>
>
> Testing
> -------
>
> See the massive test plan in the jira.
>
>
> Thanks,
>
> Jonathan Yap
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20111202/9fe3d094/attachment-0001.htm
More information about the opensource-dev
mailing list