Re: [Plplot-devel] Call for testing of the PLplot git master tip version VS2012, shared libraries

2015-03-17 Thread Alan W. Irwin
Hi laurent: On 2015-03-18 03:56+0100 laurent Berger wrote: [out of order] > With a no shared lib and with Windows 7-64 bits and MSVC 2012 Win64 bits: > //Build shared libraries > BUILD_SHARED_LIBS:BOOL=OFF > there is no error That good result makes sense because BUILD_SHARED_LIBS:BOOL=OFF means

[Plplot-devel] fixed memory leak plplot-5.10.0

2015-03-17 Thread Norman Goldstein
I noticed that at the end of plSelectDev(), in plcore.c, a couple more fields of the PLStream needed to be filled in (feedback from valgrind :-)), so I added these two lines at line 3230: plsc->tidy = (void*) dispatch_table[dev - 1]->pl_tidy; plsc->tidy_data = (void*) plsc; I expect the

Re: [Plplot-devel] Call for testing of the PLplot git master tip version VS2013

2015-03-17 Thread laurent Berger
Now I forget MSVC 2013 My cmake version is 3.1.2. I have build all versions DEBUG MinSizeRel Release RelWithDebInfo with Windows 7-64 bits and MSVC 2012 Win64 bits I have build too with Windows 7-64 bits but with MSVC 2012-32bits CMakeCache.txt is *** # EXTERNAL cache

Re: [Plplot-devel] Call for testing of the PLplot git master tip version VS2013

2015-03-17 Thread Alan W. Irwin
On 2015-03-17 22:38+0100 laurent Berger wrote: > with VS2013 like yesterday there is no problem but same error (like vs2012) > is in test_plbuf project > > 1> Building Custom Rule F:/Lib/test/plplot-plplot/examples/c/CMakeLists.txt > 1> CMake does not need to re-run because > F:\Lib\test\plplo

Re: [Plplot-devel] Call for testing of the PLplot git master tip version VS2013

2015-03-17 Thread laurent Berger
with VS2013 like yesterday there is no problem but same error (like vs2012) is in test_plbuf project 1> Building Custom Rule F:/Lib/test/plplot-plplot/examples/c/CMakeLists.txt 1> CMake does not need to re-run because F:\Lib\test\plplot-plplot\build\examples\c\CMakeFiles\generate.stamp is up-

Re: [Plplot-devel] Call for testing of the PLplot git master tip version VS2012

2015-03-17 Thread laurent Berger
with VS2012 problem is solved I can run wxPlplotdemo.exe I have seen an another error in test_plbuf 1> Building Custom Rule F:/Lib/test/plplot-plplot/examples/c/CMakeLists.txt 1> CMake does not need to re-run because F:\Lib\test\plplot-plplot\build\examples\c\CMakeFiles\generate.stamp is up-to

Re: [Plplot-devel] Call for testing of the PLplot git master tip version

2015-03-17 Thread Alan W. Irwin
On 2015-03-17 13:25-0400 Jim Dishaw wrote: On Mar 17, 2015, at 1:13 PM, "Alan W. Irwin" wrote: Your patch is based on a rather old version of src/plmetafile.c so does not apply. Please regenerate the patch based on master tip. [...]I did a "git fetch" followed by "git merge origin/master --f

Re: [Plplot-devel] Call for testing of the PLplot git master tip version

2015-03-17 Thread Jim Dishaw
On Mar 17, 2015, at 1:13 PM, "Alan W. Irwin" wrote: > Hi Jim: > > You said: > >> I think for most devices the argument will be positive. Due to the > experimental nature of plmetafile.c and the desire to change to a > different representation for internal coordinates, I don't think we > need

Re: [Plplot-devel] Call for testing of the PLplot git master tip version

2015-03-17 Thread Alan W. Irwin
Hi Jim: You said: > I think for most devices the argument will be positive. Due to the experimental nature of plmetafile.c and the desire to change to a different representation for internal coordinates, I don't think we need to worry about it for this release. Good point. >I have attached a p

Re: [Plplot-devel] Call for testing of the PLplot git master tip version

2015-03-17 Thread Jim Dishaw
0001-Fixed-linking-error-in-VS-2012-due-missing-lround-wh.patch Description: Binary data On Mar 17, 2015, at 4:10 AM, "Alan W. Irwin" wrote: > On 2015-03-16 22:08-0400 Jim Dishaw wrote: > >> Because plmetafile.c is experimental, we can swap out lround(...) > with (short)ceil(...) for now. I

Re: [Plplot-devel] Call for testing of the PLplot git master tip version

2015-03-17 Thread Alan W. Irwin
On 2015-03-16 22:08-0400 Jim Dishaw wrote: > Because plmetafile.c is experimental, we can swap out lround(...) with (short)ceil(...) for now. It appears that ceil is in VS 2012. I can code up something better after release. Hi Jim: Are any of the arguments of the lround calls in plmetafile.c gua