Hi all,
I hate to butt into a conversation here, but Jeremy is exactly correct
on this one.
Our inkscape-specific isnan.h and its ilk really need to be rolled up
into a
single .h that has all of the switches/aliases/etc for all of the
float-classification
needs that we have. It should all be grouped into a single .h so that
downstream developers don't need to worry about it. Map your needs
to Inkscape-specific names so that there is no fear of collision.
Did we not agree to this very exact thing >2 years ago?
By the way, 0.47 is looking very good. with Bulia as the warden,
how can it fail? :-)
bob
p.s.:
("butt" is the correct spelling for this metaphor. like being hit by a
male goat :-)
On 4/29/2009 4:17 PM, Jeremy C. Reed wrote:
Using inkscape-21256 snapshot (because release version has bug
causing
core dump when attempting to change fill color as reported numerous
times).
src/display/nr-arena-shape.cpp has
#include<fenv.h>
But I don't see any use of floating-point definitions,
FE* macros, or fe*() functions.
I commented that line out and inkscape built fine. (Also some platforms
don't have any fenv.h.)
Also src/live_effects/lpe-dynastroke.cpp uses fmin and fmax.
Some platforms don't have that. So I added to
src/live_effects/lpe-dynastroke.cpp :
#define fmin(a,b) ((a)<(b)?(a):(b))
#define fmax(a,b) ((a)>(b)?(a):(b))
Also work/inkscape-21256/src/display/nr-filter.cpp refers to fmax but it
is commented out.