[Opensim-dev] Simulator Statistics Testing, Planning for Patches, Next Steps

[Kevin]

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