Fix missing redraw after expose #352
Conversation
| @@ -274,6 +274,13 @@ template< class C > void EventHandler< C >::init() | |||
| } | |||
|
|
|||
| template< class C > | |||
| bool EventHandler< C >::handleEvent( eq::EventType, const eq::Event& ) | |||
| { | |||
| _impl->config.postRedraw(); | |||
There was a problem hiding this comment.
I don't know enough about Equalizer event processing, but shouldn't there be a check that this actually is an "expose event" if this is what this commit is fixing? Otherwise, it looks like this is going to trigger postRedraw more often than before. If you think this is fine, then I trust your expertise :-)
There was a problem hiding this comment.
These are events handled here, and I think all should redraw:
case EVENT_WINDOW_CLOSE:
case EVENT_WINDOW_HIDE:
case EVENT_WINDOW_EXPOSE:
case EVENT_EXIT:
case EVENT_WINDOW_POINTER_GRAB:
case EVENT_WINDOW_POINTER_UNGRAB:
There was a problem hiding this comment.
even those like CLOSE / HIDE / EXIT ? I would probably have listed made a white-list, but as long as this has no adverse side effect then I guess it's fine.
No description provided.