The variable in question is set when an error message is output, and doesn't control termination at all, so I think the new name matches the semantics better. Cc: Mitesh Shah <mshah@xxxxxxxx> Cc: Christopher Li <sparse@xxxxxxxxxxx> Cc: Jeff Garzik <jgarzik@xxxxxxxxxx> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@xxxxxxx> --- lib.c | 4 ++-- lib.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib.c b/lib.c index b4d3944..1172dc2 100644 --- a/lib.c +++ b/lib.c @@ -29,7 +29,7 @@ #include "target.h" int verbose, optimize, optimize_size, preprocessing; -int die_if_error = 0; +int error_happened = 0; #ifndef __GNUC__ # define __GNUC__ 2 @@ -131,7 +131,7 @@ void warning(struct position pos, const char * fmt, ...) static void do_error(struct position pos, const char * fmt, va_list args) { static int errors = 0; - die_if_error = 1; + error_happened = 1; show_info = 1; /* Shut up warnings after an error */ max_warnings = 0; diff --git a/lib.h b/lib.h index 2cea252..7cde9c2 100644 --- a/lib.h +++ b/lib.h @@ -25,7 +25,7 @@ #endif extern int verbose, optimize, optimize_size, preprocessing; -extern int die_if_error; +extern int error_happened; extern int repeat_phase, merge_phi_sources; extern int gcc_major, gcc_minor, gcc_patchlevel; -- 1.7.10.4 -- 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