[cairo] Re: Flag for FP-challenged platforms

[Carl Worth]

On Wed, 29 Nov 2006 16:04:32 -0800, "Daniel Amelang" wrote:
> > --avoid-floating-point and AVOID_FLOATING_POINT
...
> Hmmm...not much feedback. I'll take that to mean "looks good". I'm
> attaching a patch for final review just in case.

Well done! That's the right approach to take. I appreciate it!

>                                                  If I don't hear any
> objections, I'll then commit with my newly bestowed cairo commit
> powers!

I hope you enjoy them. You can feel like a super cow now.

> Notice that I don't try to guess what the value should be. If someone
> wants to build some smarts into this, be my guest. In the meantime,
> the embedded people will have to remember to specify
> --enable-avoid-floating-point at configure time.

Oh, yuck, now I'll complain. ;-)

I was fine to let --avoid-floating-point go by without comment but
--enable-avoid-floating-point is a pretty awkward phrase, (sounds to
me like "turning on the turn-it-off switch" ).

I suppose that autoconf is restricting us to use either --enable or
--disable here, right? Hmm.. or we can use --with too right? Though
I've never liked mixing those because I can never remember if I'm
supposed to pass --with or --enable for any given option. Add to that
the fact that autoconf's configure script silently accepts, (and
ignores), misspelled options and you've got quite a disaster on your
hands.

Or can we just cut-and-paste the implementation of AC_ARG_ENABLE into
our configure and actually advertise --avoid-floating-point as the
option name?

I've tried to come up with a phrase that works both after the word
"enable" and after the word "if", (for the macro), but didn't succeed
very well:

	--enable-floating-point-is-expensive ???
	--enable-floating-point-considered-harmful ???

-Carl
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/cairo/attachments/20061129/86312ed1/attachment-0001.pgp