DirectX 9 Timing

My work-in-progress Direct3D engine now has the option for each rendered object to have a velocity vector. I calculate the change in position right before rendering, and the timer stops and starts before and after the calculation:

Code:

	//Calc new object positions
	Timer.Stop();
	NewPositions();
	Timer.Start();

	//Render objects
	vector<CRenderObject *>::iterator i;
	for (i=Objects.begin();i!=Objects.end();i++)
	{
		Device->SetRenderState(D3DRS_FILLMODE,FillMode);
		(*i)->Render(mWorld);
	}

Code:

void DrawClass::NewPositions(void)
{
	vector<CRenderObject *>::iterator i;
	float Elapsed=(float)Timer.Elapsed();
	for (i=Objects.begin();i!=Objects.end();i++)
	{
		(*i)->x+=Elapsed*(*i)->vx;
		(*i)->y+=Elapsed*(*i)->vy;
		(*i)->z+=Elapsed*(*i)->vz;
	}
}

However, it does not work. Upon execution, each object travels only for a short period of time, eg less than one second, and then stops. If the window is manipulated, ie, moved or resized, the object moves for a little bit more. I guess that implies that the problem is related to the message loop. Here it is:

Code:

while (msg.message!=WM_QUIT)
{
	dc->Render();
	if (PeekMessage(&msg,NULL,0,0,PM_REMOVE))
	{
		TranslateMessage(&msg);
		DispatchMessage(&msg);
	}
}

As far as I can tell, it should be continually rendering, and thus updating the object positions. Is there anything that appears incorrectly coded?

The message loop I usually use is like this:

Code:

while(true) 
{
  if(PeekMessage(&msg, hwnd, NULL, NULL, PM_REMOVE))
  {
    if(msg.message == WM_QUIT)
      break;
    
    TranslateMessage(&msg);
    DispatchMessage(&msg);
  }
  else
  {
    // Do all your updates / rendering here
  }
}

I didn't look too much at your timing stuff. If this doesn't fix it then come back and post again and I'll have a look. Also, look at the STL function for_each( ) it's pretty handy. You'll need to #include <algorithm> to use it.