Re: [Qemu-devel] [PATCH 03/11] qapi: Eliminate QAPISchemaParser.__init__()'s local fname

[Marc-André Lureau]

On Mon, Oct 2, 2017 at 4:13 PM, Markus Armbruster <address@hidden> wrote:
> Signed-off-by: Markus Armbruster <address@hidden>

Reviewed-by: Marc-André Lureau <address@hidden>


> ---
>  scripts/qapi.py | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/scripts/qapi.py b/scripts/qapi.py
> index dc92bca620..6c8001e96d 100644
> --- a/scripts/qapi.py
> +++ b/scripts/qapi.py
> @@ -261,8 +261,7 @@ class QAPISchemaParser(object):
>
>      def __init__(self, fp, previously_included=[], incl_info=None):
>          abs_fname = os.path.abspath(fp.name)
> -        fname = fp.name
> -        self.fname = fname
> +        self.fname = fp.name
>          previously_included.append(abs_fname)
>          self.incl_info = incl_info
>          self.src = fp.read()
> @@ -277,7 +276,7 @@ class QAPISchemaParser(object):
>          self.accept()
>
>          while self.tok is not None:
> -            info = {'file': fname, 'line': self.line,
> +            info = {'file': self.fname, 'line': self.line,
>                      'parent': self.incl_info}
>              if self.tok == '#':
>                  self.reject_expr_doc()
> --
> 2.13.6
>
>



-- 
Marc-André Lureau