Also... I would have thought/hoped that a simplified/stable solution would eliminate "serverGeometry" completely.

/**
* Geometry retrieved from the
* last ConfigureNotify event received
*/
Geometry & geometry () const;

/**
* Geometry last sent to the server
*/
Geometry & serverGeometry () const;

I understand why we have, and why we might want, serverGeometry. However it is an optimization which only makes sense to attempt if the code is stable and bug-free to begin with.

While serious bugs remain, I think the first goal should be to simplify the logic down to just using "geometry" and remove or stub "serverGeometry".

Also... I would have thought/hoped that a simplified/stable solution would eliminate "serverGeometry" completely.

/**

* Geometry retrieved from the

* last ConfigureNotify event received

*/

Geometry & geometry () const;

/**

* Geometry last sent to the server

*/

Geometry & serverGeometry () const;

I understand why we have, and why we might want, serverGeometry. However it is an optimization which only makes sense to attempt if the code is stable and bug-free to begin with.

While serious bugs remain, I think the first goal should be to simplify the logic down to just using "geometry" and remove or stub "serverGeometry".