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.

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

Test coverage #52

Merged
merged 9 commits into from
May 24, 2017
Merged

Test coverage #52

merged 9 commits into from
May 24, 2017

Conversation

llllllllll
Copy link
Owner

Adds some test cases based on the coverage report.

Joe Jevnik added 7 commits June 27, 2016 01:38
Adds CodeTransformer.context for users to dump contextual data while
performing a transformation. This allows users to persist information
across dispatched transformation functions.

A transformation context exists for the scope of transforming a single
code object. This is reset when entering the transformation of code
object found in the consts.

By default the context will contain code object being worked on and the
startcode.

The context uses thread local storage so that a single transformer may
be used in multuiple threads at the same time.
(as an empty list).

This allows a single instance of ``CodeTransformer`` to be used
recursivly to transform code objects in a thread safe way while
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: recursively

Copy link
Collaborator

@ssanderson ssanderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One typo comment.

@llllllllll llllllllll merged commit 43df3c8 into master May 24, 2017
@llllllllll llllllllll deleted the test-coverage branch May 24, 2017 22:28
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.

None yet

2 participants