qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH] tracetool: handle DTrace keywords 'in', 'next',


From: Alon Levy
Subject: Re: [Qemu-devel] [PATCH] tracetool: handle DTrace keywords 'in', 'next', 'self'
Date: Mon, 16 Apr 2012 15:51:12 +0300
User-agent: Mutt/1.5.21 (2011-07-01)

On Mon, Apr 16, 2012 at 12:53:58PM +0100, Stefan Hajnoczi wrote:
> Language keywords cannot be used as argument names.  The DTrace backend
> appends an underscore to the argument name in order to make the argument
> name legal.
> 
> This patch adds 'in', 'next', and 'self' keywords to dtrace.py.
> 
> Also drop the unnecessary argument name lstrip() call.  The
> Arguments.build() method already ensures there is no space around
> argument names.  Furthermore it is misleading to do the lstrip() *after*
> checking against keywords because the keyword check would not match if
> spaces were in the name.
> 

Reviewed-by: Alon Levy <address@hidden>

> Signed-off-by: Stefan Hajnoczi <address@hidden>
> ---
>  scripts/tracetool/backend/dtrace.py |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/scripts/tracetool/backend/dtrace.py 
> b/scripts/tracetool/backend/dtrace.py
> index cebbd57..9cab75c 100644
> --- a/scripts/tracetool/backend/dtrace.py
> +++ b/scripts/tracetool/backend/dtrace.py
> @@ -86,10 +86,10 @@ def stap(events):
>          i = 1
>          if len(e.args) > 0:
>              for name in e.args.names():
> -                # 'limit' is a reserved keyword
> -                if name == 'limit':
> -                    name = '_limit'
> -                out('  %s = $arg%d;' % (name.lstrip(), i))
> +                # Append underscore to reserved keywords
> +                if name in ('limit', 'in', 'next', 'self'):
> +                    name += '_'
> +                out('  %s = $arg%d;' % (name, i))
>                  i += 1
>  
>          out('}')
> -- 
> 1.7.9.5
> 
> 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]