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


I don't understand this "fix" at ALL. It doesn't seem to make any sense :/... 
What you need is to set the final scale once the distance between destination 
and current become LESS than the minimum interpolation distance, because then 
interpolation will never update it anymore. This fix just makes condition true 
all the time for your test case at your frame rate, but will still fail to 
reach the final scale when you increase the frame rate (or when the destination 
scale is simply so close to the start scale that this condition is never true).

- Aleric Inglewood


On Feb. 17, 2013, 7:02 p.m., MartinRJ Fayray wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/617/
> -----------------------------------------------------------
> 
> (Updated Feb. 17, 2013, 7:02 p.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Description
> -------
> 
> At high framerates tiny prims get stuck upon interpolation when they are 
> resized via script.
> I fixed that by comparing the original scale versus the new target scale 
> (instead of comparing the original scale versus the new interpolated target 
> scale),
> in lldrawable.cpp "updateXform" to decide whether a scale change requires an 
> immediate rebuild or not.
> 
> 
> This addresses bug BUG-1709.
>     https://jira.secondlife.com/browse/BUG-1709
> 
> 
> Diffs
> -----
> 
>   indra/newview/lldrawable.cpp fbbee98b7512 
> 
> Diff: http://codereview.secondlife.com/r/617/diff/
> 
> 
> Testing
> -------
> 
> See test plan in Jira: https://jira.secondlife.com/browse/BUG-1709
> 
> Repository: https://bitbucket.org/MartinRJ/bug-1709
> 
> 
> Thanks,
> 
> MartinRJ Fayray
> 
>

_______________________________________________
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

Reply via email to