[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] tracetool: handle DTrace keywords 'in', 'next',
From: |
Lluís Vilanova |
Subject: |
Re: [Qemu-devel] [PATCH] tracetool: handle DTrace keywords 'in', 'next', 'self' |
Date: |
Mon, 16 Apr 2012 19:22:50 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1.50 (gnu/linux) |
Stefan Hajnoczi writes:
> 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.
> Signed-off-by: Stefan Hajnoczi <address@hidden>
Reviewed-by: Lluís Vilanova <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
--
"And it's much the same thing with knowledge, for whenever you learn
something new, the whole world becomes that much richer."
-- The Princess of Pure Reason, as told by Norton Juster in The Phantom
Tollbooth
- [Qemu-devel] [PATCH] tracetool: handle DTrace keywords 'in', 'next', 'self', Stefan Hajnoczi, 2012/04/16
- Re: [Qemu-devel] [PATCH] tracetool: handle DTrace keywords 'in', 'next', 'self', Alon Levy, 2012/04/16
- Re: [Qemu-devel] [PATCH] tracetool: handle DTrace keywords 'in', 'next', 'self', Stefan Hajnoczi, 2012/04/16
- Re: [Qemu-devel] [PATCH] tracetool: handle DTrace keywords 'in', 'next', 'self',
Lluís Vilanova <=