X Tutup
The Wayback Machine - https://web.archive.org/web/20260217030124/https://github.com/python/cpython/pull/1253
Skip to content

bpo-30113: Add function Allow helper function to wrap sys.setprofile#1253

Closed
louisom wants to merge 1 commit intopython:masterfrom
louisom:bpo-30113-wrap-sys-setprofile
Closed

bpo-30113: Add function Allow helper function to wrap sys.setprofile#1253
louisom wants to merge 1 commit intopython:masterfrom
louisom:bpo-30113-wrap-sys-setprofile

Conversation

@louisom
Copy link
Contributor

@louisom louisom commented Apr 22, 2017

Relate to #287, #1212.

This helper function _adjust_frame tweak the frame stack inside the profiler, it will add the helper function which wraps sys.setprofile 's frame into the profiler frame stack.

The usage:

def setprofile_helper(pr):
    pr._adjust_frame()
    sys.setprofile(pr.dispatcher)

def  profile_call(call):
    pr = profile.Profile()
    setprofile_helper(pr)
    try:
        call()
     finally:
         sys.setprofile(None)
     return pr

@mention-bot
Copy link

mention-bot commented Apr 22, 2017

@lulouie, thanks for your PR! By analyzing the history of the files in this pull request, we identified @tim-one, @nbastin and @freddrake to be potential reviewers.

@louisom louisom mentioned this pull request Apr 22, 2017
Closed
@louisom
Copy link
Contributor Author

louisom commented Apr 26, 2017

@tim-one Could you help for reviewing #1212, #287 and this code? The context of these issue are here: python-dev mailing list. Also I wrote the context manager solve method at blog post.

Hope these context will help you to review the code.
Thanks!

@louisom
Copy link
Contributor Author

louisom commented Apr 28, 2017

@arigo Could you help for reviewing these change?
Thanks!

@arigo
Copy link
Contributor

arigo commented May 1, 2017

@lulouie: sorry, I can't help with reviewing this.

@csabella
Copy link
Contributor

csabella commented Jan 20, 2020

This change is from an unknown repository and this GitHub account is no longer active. I'm going to close this PR so that another one could be opened to replace it.

@csabella csabella closed this Jan 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

awaiting review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@louisom @mention-bot @arigo @csabella @brettcannon @the-knights-who-say-ni
X Tutup