[Python-Dev] cpython: Don't use fancy new Python features like 'with' - some bots don't have them

Georg Brandl g.brandl at gmx.net
Thu Sep 26 18:07:39 CEST 2013


Am 26.09.2013 15:42, schrieb eli.bendersky:
> http://hg.python.org/cpython/rev/931d95e9067f
> changeset:   85801:931d95e9067f
> user:        Eli Bendersky <eliben at gmail.com>
> date:        Thu Sep 26 06:41:36 2013 -0700
> summary:
>   Don't use fancy new Python features like 'with' - some bots don't have them
> and can't bootstrap the parser.
> 
> files:
>   Parser/asdl.py |  5 ++++-
>   1 files changed, 4 insertions(+), 1 deletions(-)
> 
> 
> diff --git a/Parser/asdl.py b/Parser/asdl.py
> --- a/Parser/asdl.py
> +++ b/Parser/asdl.py
> @@ -398,8 +398,11 @@
>      scanner = ASDLScanner()
>      parser = ASDLParser()
>  
> -    with open(file) as f:
> +    try:
> +        f = open(file)
>          buf = f.read()
> +    finally:
> +        f.close()
>      tokens = scanner.tokenize(buf)
>      try:
>          return parser.parse(tokens)

The open call needs to go outside the try-finally.

cheers,
Georg


More information about the Python-Dev mailing list