On 2017-07-11 19:49-0000 Arjen Markus wrote:

[...]
And then it hit me: I had downloaded the _wrong_ bits
version. From there on it went quite smoothly. After some struggles
with the path I got the build to succeed and got an example to run.

I was very happy to hear that good news from you.

The only (!) problem left is that on line 378 in wxwidgets_dev.cpp the
variable "offset" is used but was not previously set.
[...]
A quick check of the source code did not tell me how to solve that, so I will 
have to leave that to you.

Indeed, I confirm that is an obvious bad case of an uninitialized
variable, and it is up to me to fix it since that code is mine.  But
it is pretty bad code in another respect (quite convoluted) so I
understand it well enough to realize there is a definite uninitialized
variable issue, but it will take me a while to figure out all the
logic again and come up with a proper fix.

By the way, this is really good news that MSVC found this issue and
you reported it because previously I was having trouble figuring out
why that code was not producing correct superscript/subscript results.
But I suspect once I fix this uninitialized variable bug, that other
problem will become understandable and solveable as well.

By the way, the correct 64-bits libraries are installed in a
directory containing the "x64" suffix. So my earlier change was (a)
not necessary (b) an indication I had downloaded the wrong libraries.
If only they had used "x86" or "x32" or something for 32-bits versions
...

So it sounds like you want to drop your previous commit to
cmake/modules/FindwxWidgets.cmake.

If that is the case, please do that immediately while you remember
these recent find troubles.  However, the quick git lesson is you
should not destroy your previous commit like you can with a private
topic branch commit that has not yet been published since the
offending commit you don't like any more is published (meaning others
should be able to count on it remaining on our public repository
indefinitely).  So instead locally reverse the changes you made to
cmake/modules/FindwxWidgets.cmake on a private topic branch. And
commit that change (with good explanatory commit message) on that
private topic branch and then merge it to master in our public
repository using our usual workflow.

So the net result is you get credit for two public commits this way;
your original one and then another separate commit to reverse it.
So it is a win-win similar to my forthcoming fix for the above
uninitialized variable error.  :-)

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
__________________________

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