Skip to content

Conversation

@icemac
Copy link
Member

@icemac icemac commented May 5, 2017

This is (nearly) a complete rewrite of the package using the Python ast module instead of using the old and un maintained compiler module.

The API was kept the same.

stephan-hof and others added 30 commits October 5, 2016 09:38
However master already supports it, so lets use master
until a official release is there.
Forbid augmented assignment of
- attributes
- subscripts
According to python2_7.ast and python3_{4,5,6}.ast these attributes are always
present. => No need for the hasattr checks.
The previous way worked great if there was a body to append the unpack statements.
Unfortunately, tuple parameters are also allowed as parameters for  lambda
functions. lambda functions don't have a body where statements are allowed :(
=> No way to use the existing code.

The new way wraps the tuple unpacking into a single expresion, by using nested
lambdas. This expression can now be used wrap tuple parameters of lambda
functions.
loechel and others added 27 commits March 27, 2017 23:36
Fix the Python-3 variant which was not equivalent to the Python-2 one.
There's a backport this package part of `future` which provides other
builtins (and breaks tests if it's installed).
Do not attempt to import builtins on Python 2.7
* base for slice tests
RestrictedPython expects it to be present in RestrictedPython.transformer.RestrictingNodeTransformer.transform_slice().
fix some spelling and conventions
Allow to create new classes with `safe_builtins` activated.
add PendingDeprecationWarnings to all old Implementation Modules
# Conflicts:
#	.gitignore
#	.travis.yml
#	docs/CHANGES.rst
#	setup.py
+ run doctest on source code
@icemac icemac requested a review from tseaver May 5, 2017 11:31
@icemac icemac merged commit 314eb03 into master May 5, 2017
@icemac icemac deleted the Python3_update branch May 10, 2017 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants