There doesn't seem to be any reason for the special member of struct token to be unsigned; AFAICT it is only ever being directly compared to explicit characters and the SPECIAL_* enum constants using ==, != and in a switch statement. Making it plain int avoids an annoying warning from match_op in token.h when compiling with -Wsign-compare. Signed-off-by: Rasmus Villemoes <rv@xxxxxxxxxxxxxxxxxx> --- token.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/token.h b/token.h index 8dbd80f..69d95d7 100644 --- a/token.h +++ b/token.h @@ -187,7 +187,7 @@ struct token { union { const char *number; struct ident *ident; - unsigned int special; + int special; struct string *string; int argnum; struct argcount count; -- 2.1.3 -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html