shithub: scc

Info  •  Files  •  Log  •  Branches

Remove redundant test of int default declarations

This check is done in in specifier, and since it is not going to
be a common case, it is better remove the redundant code and have
only the specifier version.

--- a/decl.c

+++ b/decl.c

@@ -314,14 +314,9 @@

 	struct symbol *sym;

 	switch (yytoken) {

-	case IDEN:

-		warn(options.implicit,

-		     "type defaults to 'int' in declaration");

-		tp = inttype;

-		break;

 	case SCLASS:

 		error("storage class '%s' in struct/union field", yytext);

-	case TYPE: case TQUALIFIER:

+	case IDEN: case TYPE: case TQUALIFIER:

 		tp = specifier(NULL);

 		break;

 	case ';':

@@ -470,12 +465,7 @@

 	forbid_eof = 1;

 	switch (yytoken) {

-	case IDEN:

-		warn(options.implicit,

-		     "type defaults to 'int' in declaration");

-		tp = inttype;

-		break;

-	case TYPE: case SCLASS: case TQUALIFIER:

+	case IDEN: TYPE: case SCLASS: case TQUALIFIER:

 		tp = specifier(&sclass);

 		if (sclass == REGISTER || sclass == AUTO)

 			error("incorrect storage class for file-scope declaration");