Hi Phil > Pedro can you please check if this works on your systems?
ok, I'll try it when the patch is pushed to the master . Alan I assume you are going to push the patch? I get some compiling errors on the current master, please see my last post a new idea: what about if we just override the Create() function of wxPLplotwindow ? eliminate the Show() override, and the trigger of the OnCreate function, but just simply override the Create() function. the client call would be wxPLplotwindow<wxFrame> *frame = new wxPlDemoFrame(); frame->Create( NULL, wxID_ANY, wxT( "wxPLplotDemo" ) ); frame->Show(); and the stream would be created *inside* frame->Create not posted to a queue event > PS - Pedro I just realised from your email address we are in the same > line of work. I work at the Institute for Climate and Atmospheric > Science at University of Leeds. Small world! yes, good to know ! -Pedro On 2016-12-19 09:03, Phil Rosenberg wrote: > No worries :-) > > Attached is a patch which I hope should fix things. It is the > simplest > possible fix as far as I can tell. It does have a while loop with no > counter, but unless there is some rather fatal bug in wxWidgets it > should never turn into an infinite loop. Alan if you prefer I will > add > a counter to ensure things can't go crazy. I have also added some > extra checks to wxPLplotwindow just in case the create event ends up > arriving after the first paint event or something - I don't know if > this can happen but the docs aren't clear so it does no harm to > include the checks. > > I always have to check how to create patches with git. I followed the > instructions at > > https://ariejan.net/2009/10/26/how-to-create-and-apply-a-patch-with-git/. > The appropriate apply commands are listed there too. > > Pedro can you please check if this works on your systems? > > Phil > > PS - Pedro I just realised from your email address we are in the same > line of work. I work at the Institute for Climate and Atmospheric > Science at University of Leeds. Small world! > > On 19 December 2016 at 12:51, Alan W. Irwin > <ir...@beluga.phys.uvic.ca> wrote: >> On 2016-12-19 11:31-0000 Phil Rosenberg wrote: >> >>> I'm just generating a test fix. I will send a patch round once >>> it's >>> done. If it works then I'll tidy it up. However you appear to have >>> bumped up the minimum CMake version so I need to reinstall CMake - >>> turns out the latest Windows version requires a manual uninstall >>> first. I'm sure you're not deliberately making my life hard Alan >>> ;-) >>> :-D :-D >> >> >> Sorry about that. I far prefer to make your life easy. :-) >> >> >> 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 >> __________________________ -- Pedro Vicente pedro.vice...@space-research.org http://www.space-research.org/ ------------------------------------------------------------------------------ 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
