Jacek Caban (@jacek) commented about dlls/vbscript/parser.y:
| tBYVAL Identifier EmptyBrackets_opt { $$ = new_argument_decl(ctx, $2, FALSE); }
/* these keywords may also be an identifier, depending on context */ +MemberIdentifier
- : tIdentifier { $$ = $1; }
- | tERROR { ctx->last_token = tIdentifier; $$ = $1; }
- | tEXPLICIT { ctx->last_token = tIdentifier; $$ = $1; }
- | tSTEP { ctx->last_token = tIdentifier; $$ = $1; }
The patch looks good to me now. However, please add class variables with those keywords as names to the test.