Description

ddale

opened on Jun 20, 2011

Issue body actions

Original report at SourceForge, opened Thu Mar 10 10:39:14 2011

FigureCanvasMac seems like it should be a "backend with GUI", but it doesn't implement flush_events(). Seems like it should pass, if nothing else.

SourceForge History

• On Fri Mar 25 18:54:33 2011, by pivanov314: assigned_to: 100

Activity

1. dmcdougall commented on Sep 25, 2012

   

   dmcdougall

   on Sep 25, 2012

   Member

   More actions

   I'm not sure there's a Cocoa API for this:

   http://www.cocoabuilder.com/archive/cocoa/149352-flushing-events.html
   http://cocoadev.com/wiki/FlushAppEventQueue

   It appears that event polling was phased out after OS 9. Perhaps a pass is the best option here.
2. mdehoon commented on Dec 12, 2012

   

   mdehoon

   on Dec 12, 2012

   Contributor

   More actions

   Actually why is a flush_events method needed? It is not used in the matplotlib source code. Perhaps it is better to remove flush_events altogether?
3. efiring commented on Jul 22, 2013

   

   efiring

   on Jul 22, 2013

   Member

   More actions

   @mdehoon, flush_events dates back to February, 2008 as part of the official API for gui backends, so even though we don't use it internally or in examples, we can't just rip it out. It appears to be carefully implemented in the other gui backends, and at least for them it seems to be a reasonable method to have available. Therefore I think that adding it to the macosx backend as a no-op (if indeed there is not equivalent functionality available) would be the simplest way to solve the problem and close this issue.
4. mdehoon commented on Jul 22, 2013

   

   mdehoon

   on Jul 22, 2013

   Contributor

   More actions

   OK, I'll add it then. This can be done on MacOSX using NSView displayIfNeeded.
   If somebody could please accept pull request #2233 , then I'll create a new pull request for flush_events.
5. efiring commented on Jul 23, 2013

   

   efiring

   on Jul 23, 2013

   Member

   More actions

   On 2013/07/22 1:49 PM, mdehoon wrote:

   If somebody could please accept pull request #2233
   #2233 , then I'll
   create a new pull request for flush_events.

   Done.
6. mdehoon commented on Jul 23, 2013

   

   mdehoon

   on Jul 23, 2013

   Contributor

   More actions

   @efiring Thanks. Pull request #2245 adds a flush_events method to the MacOSX backend.
7. dmcdougall commented on Aug 18, 2013

   

   dmcdougall

   on Aug 18, 2013

   Member

   More actions

   Resolved by #2245.
8. dmcdougall

   closed this as completedon Aug 18, 2013

Issue actions