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

Register test complete callback along with test #141

Merged
merged 1 commit into from Jul 28, 2015

Conversation

Projects
None yet
1 participant
@bassosimone
Member

bassosimone commented Jul 24, 2015

This should solve the problem described in issue #139, even though the solution implemented here is different from the one suggested in issue's description (but that's irrelevant).

I've tested this on Linux, using valgrind to make sure that except for the leak reported in #138 there are no further memory errors.

Closes #139.

Register test complete callback along with test
This should solve the problem described in issue #139, even though
the solution implemented here is different from the one suggested in
issue's description (but that's irrelevant).
@bassosimone

This comment has been minimized.

Show comment
Hide comment
@bassosimone

bassosimone Jul 28, 2015

Member

Tested and approved by @lorenzoPrimi, thanks!

Member

bassosimone commented Jul 28, 2015

Tested and approved by @lorenzoPrimi, thanks!

bassosimone added a commit that referenced this pull request Jul 28, 2015

Merge pull request #141 from measurement-kit/issue-139
Register test complete callback along with test

@bassosimone bassosimone merged commit 3eab9fe into master Jul 28, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@bassosimone bassosimone deleted the issue-139 branch Jul 28, 2015

bassosimone added a commit that referenced this pull request Jul 29, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment