[tools 1/2] Handle "midl_pragma warning" statement in parser
Nikolay Sivov
bunglehead at gmail.com
Tue Mar 22 14:29:21 CDT 2016
On 20.03.2016 13:11, Bolotov Sergey wrote:
> Signed-off-by: Bolotov Sergey <bolotov.s.s at yandex.ru>
> ---
> tools/widl/parser.h | 2 ++
> tools/widl/parser.l | 25 +++++++++++++++++++++++++
> tools/widl/parser.y | 38 +++++++++++++++++++++++++++++++++++++-
> tools/widl/widltypes.h | 7 +++++++
> 4 files changed, 71 insertions(+), 1 deletion(-)
>
> diff --git a/tools/widl/parser.h b/tools/widl/parser.h
> index 1bcc3c1..c15d874 100644
> --- a/tools/widl/parser.h
> +++ b/tools/widl/parser.h
> @@ -45,6 +45,8 @@ void pop_import(void);
>
> int is_type(const char *name);
>
> +int do_warning(char *toggle, warning_list_t *wnum);
> +
> extern char *temp_name;
>
> #endif
> diff --git a/tools/widl/parser.l b/tools/widl/parser.l
> index 5c2dcab..65e0f45 100644
> --- a/tools/widl/parser.l
> +++ b/tools/widl/parser.l
> @@ -164,6 +164,7 @@ UUID *parse_uuid(const char *u)
> yy_pop_state();
> }
> <PP_PRAGMA>[^\n]* parser_lval.str = xstrdup(yytext); yy_pop_state(); return aPRAGMA;
> +<INITIAL>^{ws}*"midl_pragma warning" return tPRAGMA_WARNING;
> <INITIAL,ATTR>\" yy_push_state(QUOTE); cbufidx = 0;
> <QUOTE>\" {
This is wrong, any number of spaces is allowed before 'warning'.
> yy_pop_state();
> @@ -555,3 +556,27 @@ void abort_import(void)
> for (ptr=0; ptr<import_stack_ptr; ptr++)
> unlink(import_stack[ptr].temp_name);
> }
> +
> +static void warning_disable(int warning)
> +{
> + printf("disable %d\n", warning);
> +}
> +
> +static void warning_enable(int warning)
> +{
> + printf("enable %d\n", warning);
> +}
> +
> +int do_warning(char *toggle, warning_list_t *wnum)
> +{
> + warning_t *warning;
> + if(!strcmp(toggle, "disable"))
> + LIST_FOR_EACH_ENTRY(warning, wnum, warning_t, entry)
> + warning_disable(warning->num);
> + else if(!strcmp(toggle, "enable"))
> + LIST_FOR_EACH_ENTRY(warning, wnum, warning_t, entry)
> + warning_enable(warning->num);
> + else
> + return 0;
> + return 1;
> +}
You remove this in next patch, I think it'd be enough to add parser
support first, so widl does not fail on such construct, and add handling
as 2/2.
Why do you need this in a first place? Is it to process some third party
idls?
More information about the wine-devel
mailing list