[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH RFC] libxl: support for using 'MmGg' as memory size suffix



On Sat, 2013-09-21 at 16:51 +0100, Wei Liu wrote:
> diff --git a/tools/libxl/libxlu_cfg_l.l b/tools/libxl/libxlu_cfg_l.l
> index e0ea8cf..0b1f299 100644
> --- a/tools/libxl/libxlu_cfg_l.l
> +++ b/tools/libxl/libxlu_cfg_l.l

This will need Ian J's review when he gets back.

> @@ -60,6 +60,10 @@ void xlu__cfg_yyset_column(int  column_no, yyscan_t 
> yyscanner);
>                            yylval->string= xlu__cfgl_strdup(ctx,yytext);
>                            GOT(NUMBER);
>                          }
> +[0-9][0-9a-fx]*[MmGg]   {

You copied this from above but "[0-9][0-9a-fx]*" looks odd to me. I
think it is intended to accept decimals and hex 0xNNN but it actually
ends up accepting 1111xxxxx99999ddddd and nonsensical things like that,
doesn't it?

I suppose we will reject such things later.

> +                          yylval->string= xlu__cfgl_strdup(ctx,yytext);
> +                          GOT(STRING);

This sort of subtle semantic change GOT(NUMBER)=>(STRING) is what needs
Ian's thoughts...



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.