> --- a/gateway/gwlib/octstr.c  2006-04-26 16:05:02.000000000 +0200
> +++ b/gateway/gwlib/octstr.c  2006-04-18 21:01:30.000000000 +0200
> @@ -141,7 +141,7 @@
>   * and discarding the lowest to bits to get rid of typical alignment
>   * bits.
>   */
> -#define CSTR_TO_LONG(ptr)      (labs((long) ptr) >> 2)
> +#define CSTR_TO_LONG(ptr)    (((long) ptr) >> 2)

     Someone must have changed this at some point.  I run all Xeon boxes
and rushed to check my source -- it's the "new" way you show.

#define CSTR_TO_LONG(ptr)  (((long) ptr) >> 2)

     Just pointing this out so someone doesn't at a later time add the
labs() back in -- maybe a comment should be added also that says labs()
in the above crashes on Xeon processors.  I'm running a modified version
of "cvs-20041108" if it helps.

Jon

Reply via email to