Hi Robert,

The last line in the block sets the previousTime to the currentTime.  I
think this is what causes the delta to be 0.0 on the other
PrecipitationEffect instances for most uses.  The location of ???'s
comment that pointed out the bug was misleading.

----
Tom Jolley


> -----Original Message-----
> From: Robert Osfield [mailto:[email protected]] 
> Sent: Thursday, June 11, 2009 5:29 AM
> To: OpenSceneGraph Users
> Subject: Re: [osg-users] Bug report
> 
> Hi ??
> 
> I've just done a review of the code and the static previous 
> time is only set once on initialization of the static 
> previousTime value, so if you have to separate 
> PreciptationEffect's they will both use the same previousTIme 
> value, and both will get the same delta value, which looks to 
> be appropriate to me.  You're reading that the second 
> PreciptionEffect would always get a 0 value there is not correct.
> 
> Did you actually observer a bug or is it merely your code 
> reivew made you think there was a bug?
> 
> Robert.
> _______________________________________________
> osg-users mailing list
> [email protected]
> http://lists.openscenegraph.org/listinfo.cgi/osg-users-opensce
> negraph.org
> 
_______________________________________________
osg-users mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org

Reply via email to