There's a small change that must be done in my 2nd patch

here

bool wxPLplotwindow<WXWINDOW>::Show(bool show)
{
  wxLogDebug("wxPLplotwindow::Show");
  CreateStream();
  WXWINDOW::Show(show);

}


it should be instead

return WXWINDOW::Show(show);

gcc let me get away with it, but it's an error in Visual Studio

-Pedro






----- Original Message ----- 
From: "Pedro Vicente" <pedro.vice...@space-research.org>
To: "Alan W. Irwin" <ir...@beluga.phys.uvic.ca>
Cc: "PLplot development list" <plplot-devel@lists.sourceforge.net>
Sent: Friday, December 16, 2016 1:30 PM
Subject: Re: [Plplot-devel] The pls NULL fix for some Linux platforms


> Hi Alan
>
> the patch with the 2nd commit with fix is  attached
> -Pedro
>
>
> On 2016-12-15 20:14, Alan W. Irwin wrote:
>> On 2016-12-15 18:39-0500 Pedro Vicente wrote:
>>
>>> I did what the README file said and here it is my first patch
>>> attached:-)
>>>
>>> what it has it's *just* some print messages (not my fix yet) so that
>>> you and Phil could get a better idea of the bug
>>>
>>> I have
>>>
>>> 17:57:49: Debug: wxPLplotwindow::wxPLplotwindow
>>> 17:57:49: Debug: frame->Create
>>> 17:57:49: Debug: pls NULL
>>> 17:57:49: Debug: wxPLplotwindow::RenewPlot
>>> 17:57:49: Debug: wxPLplotwindow::OnCreate
>>> 17:57:49: Debug: wxPLplotwindow::RenewPlot
>>>
>>> so this clearly shows that the stream is NULL on my linux build
>>
>> Hi Pedro:
>>
>> Your commit applied cleanly here on a topic branch.  And I like the
>> fact you have a separate commit for the debugging output and
>> later you will follow up with your commit with your actual
>> fix (for VirtualBox'd Linux platforms).
>>
>> I do have a strong suggestion that the style of your commit message
>> needs to be reformatted.
>>
>> So instead of saying
>>
>> add debug messages that show sequence of events for a bug that
>> happens
>> on linux with wxwidgets3.0 installed from packages cause is that
>> frame->Create does not trigger a OnCreate() event before the PLot()
>> call
>>
>> in your very first paragraph, you should reformat that as follows:
>>
>> wxwidgets binding: insert debug messages
>> <paragraph break>
>> Add debug messages that show sequence of events for a bug that
>> happens
>> on linux with wxwidgets3.0 installed from packages.  The cause (at
>> least on VirtualBox Linux platforms) is that frame->Create does not
>> trigger an OnCreate() event before the PLot() call as shown by the
>> following debug messages when the test_wxPLplotDemo target is built
>> on a VirtualBox'd Linux system.
>> <paragraph break>
>> 17:57:49: Debug: wxPLplotwindow::wxPLplotwindow
>> 17:57:49: Debug: frame->Create
>> 17:57:49: Debug: pls NULL
>> 17:57:49: Debug: wxPLplotwindow::RenewPlot
>> 17:57:49: Debug: wxPLplotwindow::OnCreate
>> 17:57:49: Debug: wxPLplotwindow::RenewPlot
>>
>> That first very short (one line) paragraph is important (as I also
>> noted in README.developers) because that short summary of
>> the commit is used in all sorts of places within git.
>>
>> IF you haven't done any further commits,
>> you can change to this new style by simply using
>>
>> git commit --amend
>>
>> which allows you to amend your current commit message if you have
>> nothing staged yet for your next commit.
>>
>> Or I can do a similar commit amend here (but I prefer you do it for
>> the
>> practice).
>>
>> @Phil:
>>
>> I implemented the current form of Pedro's first commit here as
>> follows:
>>
>> First update local git master to latest git master from SF as per
>> usual.  Then
>>
>> # If not there already
>> git checkout master
>> # Always develop on topic branches starting from fresh master
>> git checkout -b wxwidgets_fixes
>> # Create Pedro's commit on that new branch
>> git am
>>
>> </home/irwin/Pedro.Vincente/20161215/fix/0001-add-debug-messages-that-show-sequence-of-events-for-.patch
>>
>> That commit applied cleanly (except for a minor warning about a
>> trailing whitespace issue).
>>
>> @Pedro:
>>
>> I then got the following result here from building test_wxPLplotDemo
>>
>> 16:37:03: Debug: wxPLplotwindow::wxPLplotwindow
>> 16:37:03: Debug: frame->Create
>> 16:37:03: Debug: wxPLplotwindow::RenewPlot
>> 16:37:03: Debug: wxPLplotwindow::RenewPlot
>> 16:37:03: Debug: wxPLplotwindow::OnCreate
>> 16:37:03: Debug: wxPLplotwindow::RenewPlot
>> 16:37:03: Debug: Plot()
>> 16:37:03: Debug: wxPLplotwindow::RenewPlot
>> PLMemoryMap::close: just entering close
>> [100%] Built target test_wxPLplotDemo
>>
>> which continues to show non-NULL pls here.
>>
>> Nevertheless from my perspective as release manager we want to
>> accommodate all platforms including VirtualBox ones (even if we are
>> missing something now, as Phil suggested, to explain this difference
>> between those platforms and ordinary Linux).
>>
>> So please go ahead and send me both commits (the first one amended as
>> above, and your next commit to take care of this pls NULL issue on
>> VirtualBox'd Linux platforms). And then assuming there are no
>> problems
>> here due to these two commits, I will merge both commits to SF master
>> unless I hear differently from Phil between now and then.
>>
>> 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/


--------------------------------------------------------------------------------


> ------------------------------------------------------------------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, SlashDot.org! http://sdm.link/slashdot


--------------------------------------------------------------------------------


> _______________________________________________
> Plplot-devel mailing list
> Plplot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/plplot-devel
> 


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most 
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to