Re: [PATCH 0/5] MR2444: widl: More lexer simplification.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about tools/widl/parser.l:
> -
>                                            }
> - winrt[^\n]*                                 {
> -
>                                                yy_pop_state();
> -
>                                                winrt_enable( FALSE );
> -
>                                            }
> - [^\n]*                                      {
> -
>                                                yy_pop_state();
> -
>                                                return token_str( aPRAGMA, yytext, yylval );
> -
>                                            }
>
> +}
> +<PP_LINE>[0-9]+{ws}*                            {
> -
>                                                line_number = strtoul( yytext, NULL, 10 ) - 1;  /* We didn't read the newline */
> -
>                                                yy_pop_state();
> -
>                                                yy_push_state(PP_FILE);
> -
>                                            }
>
> +<PP_FILE>"(\.|[^"\])*"{ws}*                 {

I think that \n should not be allowed in the quoted file name.