@Alan
> It appears your proposed solution to swap the order in which Yield and > Show are called works on your CentOS platform which has always given > you the most trouble. If it works on all of them please prepare the > relevant commit ok, I just posted a question about this on the wxWidgets forum https://groups.google.com/forum/#!topic/wx-dev/wsk--AlQNzU If noboby replies in a day or 2, I''ll commit that workaround patch. It would be nice to sort this out if we can. @Phil I did a simple wxWidgets app (no PLplot) that shows the issue it's here , wx_demo.cpp https://github.com/pedro-vicente/plplot-wxwidgets I get this output (OnCreate is called after the main app tries to test frame->IsReady() ) the stream here is simulated by a boolean 18:33:43: Debug: wxFrameTest::wxFrameTest 18:33:43: Debug: frame->Create 18:33:43: Debug: wxFrameTest::Show 18:33:43: Debug: frame->Show 18:33:43: Debug: FALSE frame->IsReady 18:33:43: Debug: wxFrameTest::OnSize 18:33:43: Debug: wxFrameTest::OnCreate 18:33:43: Debug: wxFrameTest::CreateStream ----- Original Message ----- From: "Alan W. Irwin" <ir...@beluga.phys.uvic.ca> To: "Pedro Vicente" <pedro.vice...@space-research.org>; "Phil Rosenberg" <p.d.rosenb...@gmail.com>; "PLplot development list" <plplot-devel@lists.sourceforge.net> Sent: Wednesday, December 21, 2016 7:36 PM Subject: RE: [Plplot-devel] Infinite Yielding issue > On 2016-12-21 23:25-0000 p.d.rosenb...@gmail.com wrote: > >> Sorry I have not been able to look at this today. I have had family >> things on. > >> Thanks for looking into that. I half suspected that might be the > case. So after everything I said about contracts with the Show > function, it seems it is already broken ☹ > > To Pedro and Phil: > > Since we are all running out of time not only because of family activities > during this season but also because the release date is only 6 days > from now, I suggest we come up with a quick compromise fix for this > release even if you guys don't completely understand why it works. > > Here is my suggestion for such a fix. > > @Pedro: > > It appears your proposed solution to swap the order in which Yield and > Show are called works on your CentOS platform which has always given > you the most trouble. So that sounds promising for a quick fix, but > please also test your proposed solution on all platforms accessible to > you including your Windows platform (just in case a change since you > last tested that platform has somehow clobbered wxwidgets on that > platform). If it works on all of them please prepare the relevant > commit (with a tested by: paragraph listing all the platforms you > tested for future reference just in case we run into trouble with this > part of our wxwidgets-related software ever again) and send it to me > for one final test that it also works here. Assuming all those tests > work, I will push that commit, and we will be done with this bug for > at least this release! > > Alan > > __________________________ > Alan W. Irwin > > Astronomical research affiliation with Department of Physics and > Astronomy, > University of Victoria (astrowww.phys.uvic.ca). > > Programming affiliations with the FreeEOS equation-of-state > implementation for stellar interiors (freeeos.sf.net); the Time > Ephemerides project (timeephem.sf.net); PLplot scientific plotting > software package (plplot.sf.net); the libLASi project > (unifont.org/lasi); the Loads of Linux Links project (loll.sf.net); > and the Linux Brochure Project (lbproject.sf.net). > __________________________ > > Linux-powered Science > __________________________ > ------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today.http://sdm.link/intel _______________________________________________ Plplot-devel mailing list Plplot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/plplot-devel
