Commit 546c8974 by David Malcolm Committed by David Malcolm

fortran: remove trailing exclamation mark from various diagnostics (PR fortran/79852)

gcc/fortran/ChangeLog:
	PR fortran/79852
	* bbt.c (insert): Remove trailing exclamation mark from message.
	* decl.c (gfc_match_final_decl): Likewise.
	* dump-parse-tree.c (show_expr): Likewise.
	* module.c (gfc_use_module): Likewise.
	* primary.c (build_actual_constructor): Likewise.
	(gfc_convert_to_structure_constructor): Likewise.

From-SVN: r248283
parent 43e89e0a
2017-05-19 David Malcolm <dmalcolm@redhat.com>
PR fortran/79852
* bbt.c (insert): Remove trailing exclamation mark from message.
* decl.c (gfc_match_final_decl): Likewise.
* dump-parse-tree.c (show_expr): Likewise.
* module.c (gfc_use_module): Likewise.
* primary.c (build_actual_constructor): Likewise.
(gfc_convert_to_structure_constructor): Likewise.
2017-05-19 Thomas Schwinge <thomas@codesourcery.com> 2017-05-19 Thomas Schwinge <thomas@codesourcery.com>
* gfortran.h (enum gfc_omp_default_sharing): Add * gfortran.h (enum gfc_omp_default_sharing): Add
......
...@@ -116,7 +116,7 @@ insert (gfc_bbt *new_bbt, gfc_bbt *t, compare_fn compare) ...@@ -116,7 +116,7 @@ insert (gfc_bbt *new_bbt, gfc_bbt *t, compare_fn compare)
t = rotate_left (t); t = rotate_left (t);
} }
else /* if (c == 0) */ else /* if (c == 0) */
gfc_internal_error("insert_bbt(): Duplicate key found!"); gfc_internal_error("insert_bbt(): Duplicate key found");
return t; return t;
} }
......
...@@ -9969,7 +9969,7 @@ gfc_match_final_decl (void) ...@@ -9969,7 +9969,7 @@ gfc_match_final_decl (void)
for (f = block->f2k_derived->finalizers; f; f = f->next) for (f = block->f2k_derived->finalizers; f; f = f->next)
if (f->proc_sym == sym) if (f->proc_sym == sym)
{ {
gfc_error ("%qs at %C is already defined as FINAL procedure!", gfc_error ("%qs at %C is already defined as FINAL procedure",
name); name);
return MATCH_ERROR; return MATCH_ERROR;
} }
......
...@@ -573,7 +573,7 @@ show_expr (gfc_expr *p) ...@@ -573,7 +573,7 @@ show_expr (gfc_expr *p)
default: default:
gfc_internal_error gfc_internal_error
("show_expr(): Bad intrinsic in expression!"); ("show_expr(): Bad intrinsic in expression");
} }
show_expr (p->value.op.op1); show_expr (p->value.op.op1);
......
...@@ -6976,7 +6976,7 @@ gfc_use_module (gfc_use_list *module) ...@@ -6976,7 +6976,7 @@ gfc_use_module (gfc_use_list *module)
for (p = gfc_state_stack; p; p = p->previous) for (p = gfc_state_stack; p; p = p->previous)
if ((p->state == COMP_MODULE || p->state == COMP_SUBMODULE) if ((p->state == COMP_MODULE || p->state == COMP_SUBMODULE)
&& strcmp (p->sym->name, module_name) == 0) && strcmp (p->sym->name, module_name) == 0)
gfc_fatal_error ("Can't USE the same %smodule we're building!", gfc_fatal_error ("Can't USE the same %smodule we're building",
p->state == COMP_SUBMODULE ? "sub" : ""); p->state == COMP_SUBMODULE ? "sub" : "");
init_pi_tree (); init_pi_tree ();
......
...@@ -2677,7 +2677,7 @@ build_actual_constructor (gfc_structure_ctor_component **comp_head, ...@@ -2677,7 +2677,7 @@ build_actual_constructor (gfc_structure_ctor_component **comp_head,
else if (!comp->attr.artificial) else if (!comp->attr.artificial)
{ {
gfc_error ("No initializer for component %qs given in the" gfc_error ("No initializer for component %qs given in the"
" structure constructor at %C!", comp->name); " structure constructor at %C", comp->name);
return false; return false;
} }
} }
...@@ -2760,13 +2760,13 @@ gfc_convert_to_structure_constructor (gfc_expr *e, gfc_symbol *sym, gfc_expr **c ...@@ -2760,13 +2760,13 @@ gfc_convert_to_structure_constructor (gfc_expr *e, gfc_symbol *sym, gfc_expr **c
{ {
if (last_name) if (last_name)
gfc_error ("Component initializer without name after component" gfc_error ("Component initializer without name after component"
" named %s at %L!", last_name, " named %s at %L", last_name,
actual->expr ? &actual->expr->where actual->expr ? &actual->expr->where
: &gfc_current_locus); : &gfc_current_locus);
else else
gfc_error ("Too many components in structure constructor at " gfc_error ("Too many components in structure constructor at "
"%L!", actual->expr ? &actual->expr->where "%L", actual->expr ? &actual->expr->where
: &gfc_current_locus); : &gfc_current_locus);
goto cleanup; goto cleanup;
} }
...@@ -2802,7 +2802,7 @@ gfc_convert_to_structure_constructor (gfc_expr *e, gfc_symbol *sym, gfc_expr **c ...@@ -2802,7 +2802,7 @@ gfc_convert_to_structure_constructor (gfc_expr *e, gfc_symbol *sym, gfc_expr **c
if (!strcmp (comp_iter->name, comp_tail->name)) if (!strcmp (comp_iter->name, comp_tail->name))
{ {
gfc_error ("Component %qs is initialized twice in the structure" gfc_error ("Component %qs is initialized twice in the structure"
" constructor at %L!", comp_tail->name, " constructor at %L", comp_tail->name,
comp_tail->val ? &comp_tail->where comp_tail->val ? &comp_tail->where
: &gfc_current_locus); : &gfc_current_locus);
goto cleanup; goto cleanup;
...@@ -2814,7 +2814,7 @@ gfc_convert_to_structure_constructor (gfc_expr *e, gfc_symbol *sym, gfc_expr **c ...@@ -2814,7 +2814,7 @@ gfc_convert_to_structure_constructor (gfc_expr *e, gfc_symbol *sym, gfc_expr **c
&& gfc_is_coindexed (comp_tail->val)) && gfc_is_coindexed (comp_tail->val))
{ {
gfc_error ("Coindexed expression to pointer component %qs in " gfc_error ("Coindexed expression to pointer component %qs in "
"structure constructor at %L!", comp_tail->name, "structure constructor at %L", comp_tail->name,
&comp_tail->where); &comp_tail->where);
goto cleanup; goto cleanup;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment