X-Git-Url: https://git.saurik.com/bison.git/blobdiff_plain/5458913abab5e33938892b0d279f02f3b09d94db..70b7c357476ed3525ddb5d2739e70690cfebb207:/src/complain.c diff --git a/src/complain.c b/src/complain.c index 7d905de5..d43e623e 100644 --- a/src/complain.c +++ b/src/complain.c @@ -1,6 +1,6 @@ /* Declaration for error-reporting function for Bison. - Copyright (C) 2000-2002, 2004-2006, 2009-2012 Free Software + Copyright (C) 2000-2002, 2004-2006, 2009-2013 Free Software Foundation, Inc. This program is free software: you can redistribute it and/or modify @@ -28,10 +28,14 @@ #include "complain.h" #include "files.h" #include "getargs.h" +#include "quote.h" -warnings warnings_flag = Wconflicts_sr | Wconflicts_rr | Wother; +warnings warnings_flag = + Wconflicts_sr | Wconflicts_rr | Wdeprecated | Wother; -bool complaint_issued; +warnings errors_flag; + +err_status complaint_status = status_none; static unsigned *indent_ptr = 0; void @@ -45,6 +49,7 @@ warnings_print_categories (warnings warn_flags) "yacc", "conflicts-sr", "conflicts-rr", + "deprecated", "other" }; @@ -53,7 +58,9 @@ warnings_print_categories (warnings warn_flags) for (i = 0; i < ARRAY_CARDINALITY (warn_names); ++i) if (warn_flags & 1 << i) { - fprintf (stderr, "%s-W%s", any ? ", " : " [", warn_names[i]); + bool err = warn_flags & errors_flag; + fprintf (stderr, "%s-W", any ? ", " : " ["); + fprintf (stderr, "%s%s", err ? "error=" : "" , warn_names[i]); any = true; } if (any) @@ -78,17 +85,18 @@ void error_message (const location *loc, warnings flags, const char *prefix, const char *message, va_list args) { - (void) flags; unsigned pos = 0; if (loc) pos += location_print (stderr, *loc); else - pos += fprintf(stderr, "%s", current_file ? current_file : program_name); - pos += fprintf(stderr, ": "); + pos += fprintf (stderr, "%s", current_file ? current_file : program_name); + pos += fprintf (stderr, ": "); if (indent_ptr) { + if (*indent_ptr) + prefix = NULL; if (!*indent_ptr) *indent_ptr = pos; else if (*indent_ptr > pos) @@ -103,92 +111,96 @@ error_message (const location *loc, warnings flags, const char *prefix, warnings_print_categories (flags); { size_t l = strlen (message); - if (l < 2 || message[l-2] != ':' || message[l-1] != ' ') + if (l < 2 || message[l - 2] != ':' || message[l - 1] != ' ') { putc ('\n', stderr); fflush (stderr); + if (loc && feature_flag & feature_caret && !(flags & no_caret)) + location_caret (stderr, *loc); } } + fflush (stderr); } /** Raise a complaint. That can be a fatal error, a complaint or just a warning. */ - static inline void complains (const location *loc, warnings flags, const char *message, va_list args) { - if (flags & complaint) - { - error_message (loc, complaint, NULL, message, args); - complaint_issued = true; - } - else if (flags & fatal) - { - error_message (loc, fatal, _("fatal error"), message, args); - exit (EXIT_FAILURE); - } - else if (flags & Wyacc) - { - if (yacc_flag) - { - error_message (loc, flags, NULL, message, args); - complaint_issued = true; - } - else if (warnings_flag & Wyacc) - { - set_warning_issued (); - error_message (loc, flags, _("warning"), message, args); - } - } - else if (warnings_flag & flags) - { - set_warning_issued (); - error_message (loc, flags, _("warning"), message, args); - } + const char* prefix = + flags & fatal ? _("fatal error") + : flags & (errors_flag | complaint) ? _("error") + : _("warning"); + + if ((flags & complaint) && complaint_status < status_complaint) + complaint_status = status_complaint; + else if ((flags & (warnings_flag & errors_flag)) && ! complaint_status) + complaint_status = status_warning_as_error; + if (flags & (warnings_flag | fatal | complaint)) + error_message (loc, flags, prefix, message, args); + if (flags & fatal) + exit (EXIT_FAILURE); } void -complain (warnings flags, const char *message, ...) +complain (location const *loc, warnings flags, const char *message, ...) { va_list args; va_start (args, message); - complains (NULL, flags, message, args); + complains (loc, flags, message, args); va_end (args); } void -complain_at (location loc, warnings flags, const char *message, ...) +complain_indent (location const *loc, warnings flags, unsigned *indent, + const char *message, ...) { va_list args; + indent_ptr = indent; va_start (args, message); - complains (&loc, flags, message, args); + complains (loc, flags, message, args); va_end (args); } -void complain_at_indent (location loc, warnings flags, unsigned *indent, - const char *message, ...) +void +complain_args (location const *loc, warnings w, unsigned *indent, + int argc, char *argv[]) { - indent_ptr = indent; + switch (argc) + { + case 1: + complain_indent (loc, w, indent, "%s", _(argv[0])); + break; + case 2: + complain_indent (loc, w, indent, _(argv[0]), argv[1]); + break; + case 3: + complain_indent (loc, w, indent, _(argv[0]), argv[1], argv[2]); + break; + case 4: + complain_indent (loc, w, indent, _(argv[0]), argv[1], argv[2], argv[3]); + break; + case 5: + complain_indent (loc, w, indent, _(argv[0]), argv[1], argv[2], argv[3], + argv[4]); + break; + default: + complain (loc, fatal, "too many arguments for complains"); + break; + } - va_list args; - va_start (args, message); - complains (&loc, flags, message, args); - va_end (args); } -/*--------------------------------. -| Report a warning, and proceed. | -`--------------------------------*/ - void -set_warning_issued (void) +deprecated_directive (location const *loc, char const *old, char const *upd) { - static bool warning_issued = false; - if (!warning_issued && (warnings_flag & Werror)) - { - fprintf (stderr, "%s: warnings being treated as errors\n", program_name); - complaint_issued = true; - } - warning_issued = true; + if (feature_flag & feature_caret) + complain (loc, Wdeprecated, + _("deprecated directive, use %s"), + quote_n (1, upd)); + else + complain (loc, Wdeprecated, + _("deprecated directive: %s, use %s"), + quote (old), quote_n (1, upd)); }