diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog index 2e88fbf8b3e565229057917b0a2b120494b4453e..371860a95e8bb101d8148bbdd1a62cb7e58d1990 100644 --- a/gcc/fortran/ChangeLog +++ b/gcc/fortran/ChangeLog @@ -1,3 +1,11 @@ +2006-09-15 Paul Thomas <pault@gcc.gnu.org> + + PR fortran/29051 + * decl.c (match_old_style_init): Set the 'where' field of the + gfc_data structure 'newdata'. + + * match.c (match_case_eos): Add a comprehensible error message. + 2006-09-13 Wolfgang Gellerich <gellerich@de.ibm.com> * trans-expr.c (gfc_add_interface_mapping): For characters, dereference diff --git a/gcc/fortran/decl.c b/gcc/fortran/decl.c index 19bf1b0ee76c4b4f3672345ee77bf7c857aae265..6045b204663c25490dae52d07cffc30eef4c7f95 100644 --- a/gcc/fortran/decl.c +++ b/gcc/fortran/decl.c @@ -395,6 +395,7 @@ match_old_style_init (const char *name) newdata = gfc_get_data (); newdata->var = gfc_get_data_variable (); newdata->var->expr = gfc_get_variable_expr (st); + newdata->where = gfc_current_locus; /* Match initial value list. This also eats the terminal '/'. */ diff --git a/gcc/fortran/match.c b/gcc/fortran/match.c index d90fc47321091f34e7d917cda1cf664364b0ab51..3f393856798e10118608e8649acbb7a7798a76a5 100644 --- a/gcc/fortran/match.c +++ b/gcc/fortran/match.c @@ -3056,7 +3056,10 @@ match_case_eos (void) /* If the case construct doesn't have a case-construct-name, we should have matched the EOS. */ if (!gfc_current_block ()) - return MATCH_ERROR; + { + gfc_error ("Expected the name of the select case construct at %C"); + return MATCH_ERROR; + } gfc_gobble_whitespace (); diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 0e52fad14c5c86e6a52711b2c44de1241c4939cf..d2cb97d59c33f206951ac3e8e254c70ceaea9e6d 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2006-09-15 Paul Thomas <pault@gcc.gnu.org> + + PR fortran/29051 + * gfortran.dg/oldstyle_3.f90: New test. + 2006-09-15 Jerry DeLisle <jvdelisle@gcc.gnu.org> PR libgfortran/29099 diff --git a/gcc/testsuite/gfortran.dg/oldstyle_3.f90 b/gcc/testsuite/gfortran.dg/oldstyle_3.f90 new file mode 100644 index 0000000000000000000000000000000000000000..dad69568b51c89047ae9550f07b15afef7cc1238 --- /dev/null +++ b/gcc/testsuite/gfortran.dg/oldstyle_3.f90 @@ -0,0 +1,10 @@ +! { dg-do compile } +! Suppress the warning about an old-style initializer; +! { dg-options "" } +! This tests the fix for PR29052 in which the error below would cause a seg-fault +! because the locus of the initializer was never set. +! +! Contributed by Bud Davis <bdavis@gcc.gnu.org> +! + character*10 a(4,2) /'aaa','bbb','ccc','ddd'/ ! { dg-error "more variables than values" } + end