mirror of
https://git.savannah.gnu.org/git/bison.git
synced 2026-03-10 04:43:03 +00:00
This patch contains more fixes to prefer signed to unsigned integer types, as modern tools like 'gcc -fsanitize=undefined' can check for signed integer overflow but not unsigned overflow. * NEWS: Document the API change. * boostrap.conf (gnulib_modules): Add intprops. * data/skeletons/glr.c: Include stddef.h and stdint.h, since this skeleton can assume C99 or later. (YYSIZEMAX): Now signed, and the minimum of SIZE_MAX and PTRDIFF_MAX. (yybool) [!__cplusplus]: Now signed (which is how bool behaves). (YYTRANSLATE): Avoid use of unsigned, and make the macro safe even for values greater than UINT_MAX. (yytnamerr, struct yyGLRState, struct yyGLRStateSet, struct yyGLRStack) (yyaddDeferredAction, yyinitStateSet, yyinitGLRStack) (yyexpandGLRStack, yymarkStackDeleted, yyremoveDeletes) (yyglrShift, yyglrShiftDefer, yy_reduce_print, yydoAction) (yyglrReduce, yysplitStack, yyreportTree, yycompressStack) (yyprocessOneStack, yyreportSyntaxError, yyrecoverSyntaxError) (yyparse, yy_yypstack, yypstack, yypdumpstack): * tests/input.at (Torturing the Scanner): Prefer ptrdiff_t to size_t. * data/skeletons/c++.m4 (b4_yytranslate_define): * src/AnnotationList.c (AnnotationList__computePredecessorAnnotations): * src/AnnotationList.h (AnnotationIndex): * src/InadequacyList.h (InadequacyListNodeCount): * src/closure.c (closure_new): * src/complain.c (error_message, complains, complain_indent) (complain_args, duplicate_directive, duplicate_rule_directive): * src/gram.c (nritems, ritem_print, grammar_dump): * src/ielr.c (ielr_compute_ritem_sees_lookahead_set) (ielr_item_has_lookahead, ielr_compute_annotation_lists) (ielr_compute_lookaheads): * src/location.c (columns, boundary_print, location_print): * src/muscle-tab.c (muscle_percent_define_insert) (muscle_percent_define_check_values): * src/output.c (prepare_rules, prepare_actions): * src/parse-gram.y (id, handle_require): * src/reader.c (record_merge_function_type, packgram): * src/reduce.c (nuseless_productions, nuseless_nonterminals) (inaccessable_symbols): * src/relation.c (relation_print): * src/scan-code.l (variant, variant_table_size, variant_count) (variant_add, get_at_spec, show_sub_message, show_sub_messages) (parse_ref): * src/scan-gram.l (<SC_ESCAPED_STRING,SC_ESCAPED_CHARACTER>) (scan_integer, convert_ucn_to_byte, handle_syncline): * src/scan-skel.l (at_complain): * src/symtab.c (complain_symbol_redeclared) (complain_semantic_type_redeclared, complain_class_redeclared) (symbol_class_set, complain_user_token_number_redeclared): * src/tables.c (conflict_tos, conflrow, conflict_table) (conflict_list, save_row, pack_vector): * tests/local.at (AT_YYLEX_DEFINE(c)): Prefer signed to unsigned integer. * data/skeletons/lalr1.cc (yy_lac_check_): * tests/actions.at (_AT_CHECK_PRINTER_AND_DESTRUCTOR): * tests/local.at (AT_YYLEX_DEFINE(c)): Omit now-unnecessary casts. * data/skeletons/location.cc (b4_location_define): * doc/bison.texi (Mfcalc Lexer, C++ position, C++ location): Prefer int to unsigned for line and column numbers. Change example to abort explicitly on memory exhaustion, and fix an off-by-one bug that led to undefined behavior. * data/skeletons/stack.hh (stack::operator[]): Also allow ptrdiff_t indexes. (stack::pop, slice::slice, slice::operator[]): Index arg is now ptrdiff_t, not int. (stack::ssize): New method. (slice::range_): Now ptrdiff_t, not int. * data/skeletons/yacc.c (b4_state_num_type): Remove. All uses replaced by b4_int_type. (YY_CONVERT_INT_BEGIN, YY_CONVERT_INT_END): New macros. (yylac, yyparse): Use them around conversions that -Wconversion would give false alarms about. Omit unnecessary casts. (yy_stack_print): Use int rather than unsigned, and omit a cast that doesn’t seem to be needed here any more. * examples/c++/variant.yy (yylex): * examples/c++/variant-11.yy (yylex): Omit no-longer-needed conversions to unsigned. * src/InadequacyList.c (InadequacyList__new_conflict): Don’t assume *node_count is unsigned. * src/output.c (muscle_insert_unsigned_table): Remove; no longer used.
174 lines
5.6 KiB
C
174 lines
5.6 KiB
C
/* Declaration for error-reporting function for Bison.
|
|
|
|
Copyright (C) 2000-2002, 2006, 2009-2015, 2018-2019 Free Software
|
|
Foundation, Inc.
|
|
|
|
This program is free software: you can redistribute it and/or modify
|
|
it under the terms of the GNU General Public License as published by
|
|
the Free Software Foundation, either version 3 of the License, or
|
|
(at your option) any later version.
|
|
|
|
This program is distributed in the hope that it will be useful,
|
|
but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
GNU General Public License for more details.
|
|
|
|
You should have received a copy of the GNU General Public License
|
|
along with this program. If not, see <http://www.gnu.org/licenses/>. */
|
|
|
|
#ifndef COMPLAIN_H_
|
|
# define COMPLAIN_H_ 1
|
|
|
|
# include "location.h"
|
|
|
|
/* Sub-messages indent. */
|
|
# define SUB_INDENT (4)
|
|
|
|
/*---------------.
|
|
| Error stream. |
|
|
`---------------*/
|
|
|
|
/** Enable a style on \a out provided it's stderr. */
|
|
void begin_use_class (const char *style, FILE *out);
|
|
|
|
/** Disable a style on \a out provided it's stderr. */
|
|
void end_use_class (const char *style, FILE *out);
|
|
|
|
/** Flush \a out. */
|
|
void flush (FILE *out);
|
|
|
|
|
|
/*-------------.
|
|
| --warnings. |
|
|
`-------------*/
|
|
|
|
/** The bits assigned to each warning type. */
|
|
typedef enum
|
|
{
|
|
warning_conflicts_rr,
|
|
warning_conflicts_sr,
|
|
warning_deprecated,
|
|
warning_empty_rule,
|
|
warning_midrule_values,
|
|
warning_other,
|
|
warning_precedence,
|
|
warning_yacc, /**< POSIXME. */
|
|
|
|
warnings_size /**< The number of warnings. Must be last. */
|
|
} warning_bit;
|
|
|
|
/** Whether -Werror was set. */
|
|
extern bool warnings_are_errors;
|
|
|
|
void warning_usage (FILE *out);
|
|
|
|
/** Decode a single argument from -W.
|
|
*
|
|
* \param arg the subarguments to decode.
|
|
* If null, then activate all the flags.
|
|
* \param no length of the potential "no-" prefix.
|
|
* Can be 0 or 3. If 3, negate the action of the subargument.
|
|
* \param err length of a potential "error=".
|
|
* Can be 0 or 6. If 6, treat the subargument as a CATEGORY.
|
|
*
|
|
* If VALUE != 0 then KEY sets flags and no-KEY clears them.
|
|
* If VALUE == 0 then KEY clears all flags from \c all and no-KEY sets all
|
|
* flags from \c all. Thus no-none = all and no-all = none.
|
|
*/
|
|
void warning_argmatch (char const *arg, size_t no, size_t err);
|
|
|
|
/** Decode a comma-separated list of arguments from -W.
|
|
*
|
|
* \param args comma separated list of effective subarguments to decode.
|
|
* If 0, then activate all the flags.
|
|
*/
|
|
void warnings_argmatch (char *args);
|
|
|
|
|
|
/*-----------.
|
|
| complain. |
|
|
`-----------*/
|
|
|
|
/** Initialize this module. */
|
|
void complain_init (void);
|
|
|
|
/** Reclaim resources. */
|
|
void complain_free (void);
|
|
|
|
/** Initialize support for colored messages. */
|
|
void complain_init_color (void);
|
|
|
|
typedef enum
|
|
{
|
|
Wnone = 0, /**< Issue no warnings. */
|
|
|
|
Wconflicts_rr = 1 << warning_conflicts_rr,
|
|
Wconflicts_sr = 1 << warning_conflicts_sr,
|
|
Wdeprecated = 1 << warning_deprecated,
|
|
Wempty_rule = 1 << warning_empty_rule,
|
|
Wmidrule_values = 1 << warning_midrule_values,
|
|
Wother = 1 << warning_other,
|
|
Wprecedence = 1 << warning_precedence,
|
|
Wyacc = 1 << warning_yacc,
|
|
|
|
complaint = 1 << 11, /**< All complaints. */
|
|
fatal = 1 << 12, /**< All fatal errors. */
|
|
silent = 1 << 13, /**< Do not display the warning type. */
|
|
no_caret = 1 << 14, /**< Do not display caret location. */
|
|
|
|
/**< All above warnings. */
|
|
Weverything = ~complaint & ~fatal & ~silent,
|
|
Wall = Weverything & ~Wyacc
|
|
} warnings;
|
|
|
|
/** Whether the warnings of \a flags are all unset.
|
|
(Never enabled, never disabled). */
|
|
bool warning_is_unset (warnings flags);
|
|
|
|
/** Whether warnings of \a flags should be reported. */
|
|
bool warning_is_enabled (warnings flags);
|
|
|
|
/** Make a complaint, with maybe a location. */
|
|
void complain (location const *loc, warnings flags, char const *message, ...)
|
|
__attribute__ ((__format__ (__printf__, 3, 4)));
|
|
|
|
/** Likewise, but with an \a argc/argv interface. */
|
|
void complain_args (location const *loc, warnings w, int *indent,
|
|
int argc, char *arg[]);
|
|
|
|
/** Make a complaint with location and some indentation. */
|
|
void complain_indent (location const *loc, warnings flags, int *indent,
|
|
char const *message, ...)
|
|
__attribute__ ((__format__ (__printf__, 4, 5)));
|
|
|
|
|
|
/** GNU Bison extension not valid with POSIX Yacc. */
|
|
void bison_directive (location const *loc, char const *directive);
|
|
|
|
/** Report an obsolete syntax, suggest the updated one. */
|
|
void deprecated_directive (location const *loc,
|
|
char const *obsolete, char const *updated);
|
|
|
|
/** Report a repeated directive. */
|
|
void duplicate_directive (char const *directive,
|
|
location first, location second);
|
|
|
|
/** Report a repeated directive for a rule. */
|
|
void duplicate_rule_directive (char const *directive,
|
|
location first, location second);
|
|
|
|
/** Warnings treated as errors shouldn't stop the execution as regular
|
|
errors should (because due to their nature, it is safe to go
|
|
on). Thus, there are three possible execution statuses. */
|
|
typedef enum
|
|
{
|
|
status_none, /**< No diagnostic issued so far. */
|
|
status_warning_as_error, /**< A warning was issued (but no error). */
|
|
status_complaint /**< An error was issued. */
|
|
} err_status;
|
|
|
|
/** Whether an error was reported. */
|
|
extern err_status complaint_status;
|
|
|
|
#endif /* !COMPLAIN_H_ */
|