Found with the Coverity scanner. It also complains about the
fall-throughs. Either the comment will silence the warning* or at least
it helps human reviewers.
Committed as Rev. 197969 after build+regtesting on x86-64-gnu-linux.
Tobias
(* Well, one can also silence the Coverity diagnostic for that spot.)
2013-04-15 Tobias Burnus <bur...@net-b.de>
* list_read.c (finish_separator): Initialize variable.
diff --git a/libgfortran/io/list_read.c b/libgfortran/io/list_read.c
index b29fdcd..c8a1bdfc 100644
--- a/libgfortran/io/list_read.c
+++ b/libgfortran/io/list_read.c
@@ -393,7 +393,7 @@ static int
finish_separator (st_parameter_dt *dtp)
{
int c;
- int err;
+ int err = LIBERROR_OK;
restart:
eat_spaces (dtp);
@@ -433,7 +433,7 @@ finish_separator (st_parameter_dt *dtp)
return err;
goto restart;
}
-
+ /* Fall through. */
default:
unget_char (dtp, c);
break;
@@ -2788,6 +2788,7 @@ nml_get_obj_data (st_parameter_dt *dtp, namelist_info **pprev_nl,
"namelist not terminated with / or &end");
goto nml_err_ret;
}
+ /* Fall through. */
case '/':
dtp->u.p.input_complete = 1;
return true;