diff options
Diffstat (limited to 'gcc/c')
| -rw-r--r-- | gcc/c/ChangeLog | 11 | ||||
| -rw-r--r-- | gcc/c/c-decl.c | 8 | ||||
| -rw-r--r-- | gcc/c/c-parser.c | 10 | ||||
| -rw-r--r-- | gcc/c/c-typeck.c | 2 |
4 files changed, 21 insertions, 10 deletions
diff --git a/gcc/c/ChangeLog b/gcc/c/ChangeLog index 774e580..2d99155 100644 --- a/gcc/c/ChangeLog +++ b/gcc/c/ChangeLog @@ -1,3 +1,14 @@ +2016-08-30 David Malcolm <dmalcolm@redhat.com> + + * c-decl.c (implicit_decl_warning): Use add_fixit_replace + rather than add_fixit_misspelled_id. + (undeclared_variable): Likewise. + * c-parser.c (c_parser_declaration_or_fndef): Likewise. Remove + now-redundant "here" params from add_fixit_insert method calls. + (c_parser_parameter_declaration): Likewise. + * c-typeck.c (build_component_ref): Remove now-redundant range + param from add_fixit_replace method calls. + 2016-08-25 Marek Polacek <polacek@redhat.com> * c-typeck.c (parser_build_binary_op): Pass LHS to diff --git a/gcc/c/c-decl.c b/gcc/c/c-decl.c index 0c52a64..8f49c35 100644 --- a/gcc/c/c-decl.c +++ b/gcc/c/c-decl.c @@ -3096,7 +3096,7 @@ implicit_decl_warning (location_t loc, tree id, tree olddecl) if (hint) { gcc_rich_location richloc (loc); - richloc.add_fixit_misspelled_id (loc, hint); + richloc.add_fixit_replace (hint); warned = pedwarn_at_rich_loc (&richloc, OPT_Wimplicit_function_declaration, "implicit declaration of function %qE; did you mean %qs?", @@ -3109,7 +3109,7 @@ implicit_decl_warning (location_t loc, tree id, tree olddecl) if (hint) { gcc_rich_location richloc (loc); - richloc.add_fixit_misspelled_id (loc, hint); + richloc.add_fixit_replace (hint); warned = warning_at_rich_loc (&richloc, OPT_Wimplicit_function_declaration, G_("implicit declaration of function %qE;did you mean %qs?"), @@ -3437,7 +3437,7 @@ undeclared_variable (location_t loc, tree id) if (guessed_id) { gcc_rich_location richloc (loc); - richloc.add_fixit_misspelled_id (loc, guessed_id); + richloc.add_fixit_replace (guessed_id); error_at_rich_loc (&richloc, "%qE undeclared here (not in a function);" " did you mean %qs?", @@ -3455,7 +3455,7 @@ undeclared_variable (location_t loc, tree id) if (guessed_id) { gcc_rich_location richloc (loc); - richloc.add_fixit_misspelled_id (loc, guessed_id); + richloc.add_fixit_replace (guessed_id); error_at_rich_loc (&richloc, "%qE undeclared (first use in this function);" diff --git a/gcc/c/c-parser.c b/gcc/c/c-parser.c index fe0c95f..c245e70 100644 --- a/gcc/c/c-parser.c +++ b/gcc/c/c-parser.c @@ -1685,7 +1685,7 @@ c_parser_declaration_or_fndef (c_parser *parser, bool fndef_ok, if (tag_exists_p (RECORD_TYPE, name)) { /* This is not C++ with its implicit typedef. */ - richloc.add_fixit_insert (here, "struct"); + richloc.add_fixit_insert ("struct"); error_at_rich_loc (&richloc, "unknown type name %qE;" " use %<struct%> keyword to refer to the type", @@ -1693,7 +1693,7 @@ c_parser_declaration_or_fndef (c_parser *parser, bool fndef_ok, } else if (tag_exists_p (UNION_TYPE, name)) { - richloc.add_fixit_insert (here, "union"); + richloc.add_fixit_insert ("union"); error_at_rich_loc (&richloc, "unknown type name %qE;" " use %<union%> keyword to refer to the type", @@ -1701,7 +1701,7 @@ c_parser_declaration_or_fndef (c_parser *parser, bool fndef_ok, } else if (tag_exists_p (ENUMERAL_TYPE, name)) { - richloc.add_fixit_insert (here, "enum"); + richloc.add_fixit_insert ("enum"); error_at_rich_loc (&richloc, "unknown type name %qE;" " use %<enum%> keyword to refer to the type", @@ -1712,7 +1712,7 @@ c_parser_declaration_or_fndef (c_parser *parser, bool fndef_ok, const char *hint = lookup_name_fuzzy (name, FUZZY_LOOKUP_TYPENAME); if (hint) { - richloc.add_fixit_misspelled_id (here, hint); + richloc.add_fixit_replace (hint); error_at_rich_loc (&richloc, "unknown type name %qE; did you mean %qs?", name, hint); @@ -3864,7 +3864,7 @@ c_parser_parameter_declaration (c_parser *parser, tree attrs) if (hint) { gcc_rich_location richloc (token->location); - richloc.add_fixit_misspelled_id (token->location, hint); + richloc.add_fixit_replace (hint); error_at_rich_loc (&richloc, "unknown type name %qE; did you mean %qs?", token->value, hint); diff --git a/gcc/c/c-typeck.c b/gcc/c/c-typeck.c index 2f8d611..5194027 100644 --- a/gcc/c/c-typeck.c +++ b/gcc/c/c-typeck.c @@ -2474,7 +2474,7 @@ build_component_ref (location_t loc, tree datum, tree component, where the user has confused "." vs "->". */ rich_location richloc (line_table, loc); /* "loc" should be the "." token. */ - richloc.add_fixit_replace (source_range::from_location (loc), "->"); + richloc.add_fixit_replace ("->"); error_at_rich_loc (&richloc, "%qE is a pointer; did you mean to use %<->%>?", datum); |