Hi Judson,

On 3 December 2012 22:37, Judson Weissert <[email protected]> wrote:

> I was using the osgViewer::ScreenCaptureHandler today for the first time.
> While I was trying to disable the continuous capture feature, I stumbled
> upon what I believe to be a bug. The switch statement case for
> osgGA::GUIEventAdapter::FRAME is missing a break statement, so the KEYUP
> statements are also executed.
>

Thanks for spotting this error.  I've reviewed the code and added the
missing break and checked it into svn/trunk, I don't believe it'll effect
behaviour though as the executing the subsequent KEYUP block as the if
statements in that block will ensure that nothing happens.


>
> Also, I was wondering if it would be a reasonable enhancement to disable,
> for example, the continuous capture feature by simply setting the _keyEvent
> (e.g., _keyEventToggleContinuousCapture) to zero, and adding the feature
> guard to getUsage() et. al? This probably applies to other event handlers
> (such as the stats handler or any other handler with multiple key mappings).
>

If you want to disable an event handler then the best way would be to
simply remove it.

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

Reply via email to