On 2014-05-20 12:21-0700 Alan W. Irwin wrote:

> Hi Arjen:
>
> Please test revision 13125 on Cygwin.
>
> That revision changes a -debug printf specifier for size_t from %zu
> (the c99 standard) to %lu for the Microsoft case as specified by the
> Microsoft documentation of how size_t values must be formatted (since
> they don't support %zu).
>
> This change fixes a bug for the "MinGW Makefiles" case where the
> -debug option was giving gibberish for the length of filename strings.
>
> However, I am not sure this code will work properly on Cygwin.  Does
> %zu work for that platform of %lu or both?  Is the WIN32 macro #defined
> on Cygwin?  Instead of answering those questions individually, it might
> be better to simply test build-tree results on Cygwin as follows:
>
> examples/c/x00c -dev psc -debug -o test.ps
>
> For the MinGW/Wine case ("MinGW Makefiles" generator) I get
> the following debugging output from that command:
>
> plLoadDriver: Device not loaded!
> plLoadDriver: tag=psc, drvidx=0
> plLoadDriver: Trying to load ps on ps
> plGetName: Maximum length of full pathname of file to be found is 116
> plGetName: Full pathname of file to be found is 
> z:/home/wine/newstart/build_script/build_dir-1.6.1_mingw/epa_build/Source/build_plplot\data\cmap0_default.pal
> plLibOpenPdfstr: Found file 
> z:/home/wine/newstart/build_script/build_dir-1.6.1_mingw/epa_build/Source/build_plplot\data\cmap0_default.pal
> [...]
>
> Before the bug fix that 116 was gibberish in the MinGW case since
> Microsoft does not recognize the c99 %zu format specifier for size_t.
>
> If the Cygwin build has no compiler warnings for plGetName and that
> test on Cygwin also gives a reasonable number for the maximum pathname
> length (normally 7 larger than the actual pathname length), then no
> further fiddling has to be done with the plGetName function defined in
> src/plctrl.c.
>
> I don't think specific tests have to be done for the MSVC case since
> there is no question in that case that the WIN32 macro is #defined
> and the %lu format specifier used in that case should work just
> like it currently does for the MinGW case.

Hi Arjen (and others here with access to either/both Cygwin and/or MSVC):

I similarly request testing of revision 13126 by executing the same
command as above in the build tree and also in the install tree.  If
the revised code in the function plInBuildTree defined in src/plcore.c
is correct, the build-tree version (as in the results above) should
find source-tree versions of all data files such as
z:/home/wine/newstart/build_script/build_dir-1.6.1_mingw/epa_build/Source/build_plplot\data\cmap0_default.pal
above.  Also, the install-tree version should find install-tree
versions of those files.  Note at the present time I have only done
build-tree testing of revision 13126 on MinGW/Wine (as above) and
Linux, but I plan to follow that up with both build-tree and
install-tree testing on Linux, MinGW/Wine, and MinGW/MSYS/Wine.  My
Linux platform testing will probably be sufficient for all Unix
platforms including Mac OS X, but I request similar tests of revision
13126 for the Cygwin and MSVC platforms (both of which are unavailable
to me and which may respond in unanticipated ways to the code updates
I have done for plInBuildTree).

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
__________________________

------------------------------------------------------------------------------
"Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE
Instantly run your Selenium tests across 300+ browser/OS combos.
Get unparalleled scalability from the best Selenium testing platform available
Simple to use. Nothing to install. Get started now for free."
http://p.sf.net/sfu/SauceLabs
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to