Merge pull request #432 from jbampton/remove-unneeded-break

BaseMax · web-flow · commit bda311fcac60 · 2023-01-31T15:24:21.000Z
C: remove unneeded `break` statements
diff --git a/src/parser/lexer/lexer.c b/src/parser/lexer/lexer.c
@@ -137,13 +137,11 @@ Token* lexer_identifier()
 		if (keywords[j].type == TOKEN_VALUE_IDENTIFIER) // it's end of the Keyword list/table
 		{
 			return token_make_value(TOKEN_VALUE_IDENTIFIER, tmp_str);
-			break;
 		}
 		else if (keywords[j].length == ident_length // fast search performance
 			 && strncmp(keywords[j].identifier, tmp_str, keywords[j].length) == 0)
 		{
 			return token_make(keywords[j].type);
-			break;
 		}
 	}
 	//	return token_make(TOKEN_VALUE_IDENTIFIER);

Original file line number	Diff line number	Diff line change
`@@ -137,13 +137,11 @@ Token* lexer_identifier()`
`137`	`137`	`if (keywords[j].type == TOKEN_VALUE_IDENTIFIER) // it's end of the Keyword list/table`
`138`	`138`	`{`
`139`	`139`	`return token_make_value(TOKEN_VALUE_IDENTIFIER, tmp_str);`
`140`		`- break;`
`141`	`140`	`}`
`142`	`141`	`else if (keywords[j].length == ident_length // fast search performance`
`143`	`142`	`&& strncmp(keywords[j].identifier, tmp_str, keywords[j].length) == 0)`
`144`	`143`	`{`
`145`	`144`	`return token_make(keywords[j].type);`
`146`		`- break;`
`147`	`145`	`}`
`148`	`146`	`}`
`149`	`147`	`// return token_make(TOKEN_VALUE_IDENTIFIER);`