[gpm]some cleanup

[Alessandro Rubini]

> translates into
>     ((count)==0 && noneofthem);
> which just plain makes no sense as a line of C code (thus the compiler
> warning).

Ok, I didn't know it spit a warning. I though it was used as
an expression, actually. 

Srry, I've been to quick in commenting :)

> I'm voting for replacing the CHECKFAIL define with simply calling
>    if (count==0) noneofthem();
> in every case where it is used now.

It might be best, actually.