Current step function may not deliver 30 frames a second

[dash-rai]

void step(){
  float m=glutGet(GLUT_ELAPSED_TIME)/(1.0/30.0);
  if(floor(m)==m){//If the time is a multiple of 1/30 th of a second, redisplay
    world->Step(1.0/30.0,8,3);
    glutPostRedisplay();
  }
}

Worst Case: The function may execute in a way that the elapsed time is never a multiple of 1/30th of a second.
1/30th of a second = ~33ms
step() maybe called once at elapsed time = 31ms and then at 34ms.
Redisplay will not be called in that case. This may cause a drop in refresh rates.

Also, wouldn't if (!(glutGet(GLUT_ELAPSED_TIME) % 33)) be a better solution if the lowered refresh rate is a non-issue?

[upman]

Nice Spot! I meant to fix that and never came around to it.
Need a system to embed TODO's as comments :P

Not having that if condition and just calling step on the idle event seems to work fine.
Although we'll be giving up control on the number of frames per second.
This seems to work properly without stepping too fast because I think glut's event loop iterates in fixed intervals, I'm not really sure at this point. Need to investigate this further.