egcs-980129, fixed ambiguous `else' warnings in the gcc/f/ dir.

Jeffrey A Law law@cygnus.com
Fri Feb 13 02:04:00 GMT 1998


  In message < 199802112157.QAA27162@caip.rutgers.edu >you write:
  > 	This patch cleans up the ambiguous `else' warnings in the
  > egcs/gcc/f/ directory.
Thanks.  I've installed the patch.

As I mentioned in a previous message, if the if-else warning is giving
false positives, then that's a bug and we should put a test in the testsuite
for it -- not leave in non-conforming code in the g77 backend.

Additionally, most (all?) of the changes I saw in this patch fixed
the exact kind of problem that that warning was supposed to catch ie

	if
	 if
	 else

Now, the C language says exactly how this is supposed to be parsed,  so
it is not ambigious, but it *is* poor practice to write such code and
is a violation of the GNU coding standards.  I believe this is the case
that the if-else warning is supposed to be catching...


jeff



More information about the Gcc mailing list