Re: [Plplot-devel] The pls NULL fix for some Linux platforms

[Pedro Vicente]

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:-)

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

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

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


on linux with wxwidgets3.0 installed from packages cause is that


in your very first paragraph, you should reformat that as follows:

wxwidgets binding: insert debug messages
<paragraph break>

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.


practice).

@Phil:



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

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


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
__________________________

add debug messages that show sequence of events for a bug that happens frame->Create does not trigger a OnCreate() event before the PLot() call Add debug messages that show sequence of events for a bug that happens Or I can do a similar commit amend here (but I prefer you do it for the I implemented the current form of Pedro's first commit here as follows: VirtualBox'd Linux platforms). And then assuming there are no problems Astronomical research affiliation with Department of Physics and Astronomy,

--
Pedro Vicente
pedro.vice...@space-research.org
http://www.space-research.org/

0001-wxwidgets-binding-workaround-fix-for-delayed-OnCreat.patch.tar.gz
Description: Binary data

------------------------------------------------------------------------------
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