On 2015-04-26 07:28, Maxwell, Douglas CIV USARMY ARL (US) wrote:
Patch 1: Simulator FPS, Physics FPS, Total Frame Time, Physics Frame
Time, and Simulation Time.
My main computer is down and in need of repair or upgrade so I can't
test out the suggested code changes at this time. The one thing I wanted
to comment on is the use of a hard-coded variable in the patch. There is
a value of 10 in the patch that is buried in the code. I would suggest
you consider defining a variable at the top of the file to hold the
value. Should the value ever need to be changed, or needed in some other
file, it would be easier to update or use it if it was more readily
visible near the top of the file.
I have some other comments I was going to make but they will have to
wait as I have to go out in just a few minutes.
--
Cheers!
Kevin.
http://www.ve3syb.ca/ |"Nerds make the shiny things that
distract
Owner of Elecraft K2 #2172 | the mouth-breathers, and that's why
we're
| powerful!"
#include <disclaimer/favourite> | --Chris Hardwick
_______________________________________________
Opensim-dev mailing list
[email protected]
http://opensimulator.org/cgi-bin/mailman/listinfo/opensim-dev
