Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cleanup/modernize code #5

Open
pfalcon opened this issue Jan 30, 2019 · 2 comments
Open

Cleanup/modernize code #5

pfalcon opened this issue Jan 30, 2019 · 2 comments

Comments

@pfalcon
Copy link
Owner

pfalcon commented Jan 30, 2019 •
edited
Loading

The project has some dead code. That's definitely true with my refactorings, but probably was true even with code as was in CPython2.

Also, non-canonical use of constructors, etc. is there, brush it up/rewrite.

While doing this still consider compatibility with Python2.7 for extra points.
@pfalcon pfalcon mentioned this issue Feb 2, 2019
Open
@pfalcon
Copy link
Owner Author

pfalcon commented Feb 6, 2019

Ok, in pfalcon/python-compiler@51a840a, older no longer used AST-related modules are removed.

@pfalcon
Copy link
Owner Author

pfalcon commented Feb 7, 2019

Good progress on removing dead code today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@pfalcon