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 guaranteed
to always be positive?  If so, then substituting ceil for lround
should give identical results, but if not then that substitution
would introduce an off-by-one difference.

Anyhow, I have a lot on my plate at the moment so I would appreciate
you investigating the above question (if you haven't done that
already) and sending me the patch that you think should be applied.

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
__________________________

------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the 
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to