Hi JS,

On Mon, Jun 7, 2010 at 4:06 PM, Jean-Sébastien Guay
<[email protected]> wrote:
> Hi Robert,
>
> Currently, if a GraphicsWindowWin32 is created with traits->useCursor =
> false, it still shows the cursor. An app must call gw->useCursor(false)
> after having created the context as a workaround, but I think what we ask
> for in the traits should be honored... Attached is a simple fix for this.

Thanks JS, change now merged and submitted to svn/trunk.

> Also, just an implementation note, shouldn't things like this be done in the
> GraphicsWindow base class constructor? It seems to me like all
> GraphicsWindow* class will do pretty much the same things (especially since
> the called methods are virtual).

There is the issue of order of execution that you'd have to be careful
about when setting up any subclass state so in general I'd suggest
it's safer to late the subclasses do the right thing by their own
needs.

Robert.
_______________________________________________
osg-submissions mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org

Reply via email to