> 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.