On 11/29/11 13:38, Jim Meyering wrote: > -#define TYPE_IS_INTEGER(t) ((t) 1.5 == 1) > +#define TYPE_IS_INTEGER(t) ((t) 1.5F == 1)
I'd rather omit this. The constant is represented exactly and is an immediate operand of a cast. (And I wouldn't be surprised if some compilers warned about the "F" for that reason....) If GCC is warning about this, that's more of a GCC bug than anything else. I second Eric's other comments about the changes.