Stepan Kasal <[EMAIL PROTECTED]> writes:

        * lib/autoconf/general.m4 (_AC_COMPUTE_INT_COMPILE): Do not use
        "break" in the parameters to _AC_COMPILE_IFELSE; reorganize the
        code, so that _AC_COMPILE_IFELSE is called only once.

Wow, what a (ahem) clever patch!

A couple things.  First, since most hosts are two's complement, it's
better to start off with a test < 0 rather than a test > 0, since the
former test neatly divides the possible values into two equal-sized
parts.  Could you rewrite it to do that?  (This may force you to
recheck the code, which is probably a good thing here...)

Second, a minor thing:

  case $ac_lo:$ac_hi in
    :* | *:) ;;
    *)
      ...
  esac

This is slightly clearer:

  case $ac_lo:$ac_hi in
    ?*:?*)
      ...
  esac


Reply via email to