Kieran Colford wrote:
+#define STRNEQ(X, Y) (strcmp (X, Y) != 0)
1. This name is poorly chosen (it looks too much like "strncmp", which means something quite different) and it's not needed (people can just use "!STREQ").
2. STREQ should be an inline function, not a macro. There's little point to making it a macro.
3. STREQ's name should be lower case.