On 2008-07-11 12:33-0700 Alan W. Irwin wrote:

> Hi Jerry:
>
> Ada examples 9, 16, and 22 no longer work with error messages of respectively
>
> raised CONSTRAINT_ERROR : plplot_thin.adb:199 index check failed
> raised CONSTRAINT_ERROR : plplot_thin.adb:200 index check failed
> raised CONSTRAINT_ERROR : plplot_thin.adb:323 index check failed
>
> These errors must be due to some Ada interface change(s) that you have done
> fairly recently since these examples were working for me before I did an svn
> update this morning.

Hi Jerry.  Your commit of revision 8516 fixed these problems.

However, I still have concerns.  The fudging of the rounding you put back in
may not work on all platforms to take care of the index check errors for
these examples, and may not even be effective on my platform for other uses
of PLplot. For example, an application might by accident be even closer to
the limits than the standard examples or might be at the other end of the
index range (where you would have to add slightly to the rounding rather
than subtract to get the value inside the limit).

Could you give some background about why the index check is there at all? Is
it an actual integer range check or a floating-point range check? If it is
the former, I guess you have to deal with it at the Ada interface level, but
for the latter (e.g., the color map1 index range from 0. to 1.), then it
would probably be better to let the C library handle out-of-range errors in
a consistent way.  If some language interface with different rounding got
slightly beyond the floating-point limits, we could make them fuzzy in the C
library.  What I mean by that is if the actual value was outside the limit
by a small amount (say 1.e-6 in the case of the color map1 range limit),
then the C library could convert it to the limit value.

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); PLplot scientific plotting software
package (plplot.org); 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
__________________________

-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
Plplot-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to