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

trace function not set, causing some Pdb commands to fail #58933

Closed
xdegaye mannequin opened this issue May 5, 2012 · 5 comments
Closed

trace function not set, causing some Pdb commands to fail #58933

xdegaye mannequin opened this issue May 5, 2012 · 5 comments
Assignees
Labels
stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error

Comments

@xdegaye
Copy link
Mannequin

xdegaye mannequin commented May 5, 2012

BPO 14728
Nosy @loewis, @birkenfeld, @terryjreedy, @orsenthil, @asvetlov, @xdegaye
Files
  • pdb.patch
  • pdb_default.patch
  • pdb_2.7.patch
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = None
    closed_at = None
    created_at = <Date 2012-05-05.10:43:44.908>
    labels = ['type-bug', 'library']
    title = 'trace function not set, causing some Pdb commands to fail'
    updated_at = <Date 2012-12-05.11:07:49.874>
    user = 'https://github.com/xdegaye'

    bugs.python.org fields:

    activity = <Date 2012-12-05.11:07:49.874>
    actor = 'asvetlov'
    assignee = 'none'
    closed = False
    closed_date = None
    closer = None
    components = ['Library (Lib)']
    creation = <Date 2012-05-05.10:43:44.908>
    creator = 'xdegaye'
    dependencies = []
    files = ['25459', '25463', '25464']
    hgrepos = []
    issue_num = 14728
    keywords = ['patch']
    message_count = 5.0
    messages = ['159985', '159997', '159998', '160417', '176277']
    nosy_count = 7.0
    nosy_names = ['loewis', 'georg.brandl', 'terry.reedy', 'orsenthil', 'asvetlov', 'neologix', 'xdegaye']
    pr_nums = []
    priority = 'normal'
    resolution = None
    stage = 'patch review'
    status = 'open'
    superseder = None
    type = 'behavior'
    url = 'https://bugs.python.org/issue14728'
    versions = ['Python 2.7', 'Python 3.2', 'Python 3.3']

    Linked PRs

    @xdegaye
    Copy link
    Mannequin Author

    xdegaye mannequin commented May 5, 2012

    The bpo-13183 raises the problem that when the trace function is not
    set in the caller frame, the step command fails at a return statement.

    This new issue raises the point that, for the same reason:

    * the next, until and return statements fail also at a return
      statement when the caller frame does not have a trace function
    
    * when the user runs the up and down commands at any line in a
      frame to select a new frame, then the next, until or return
      commands fail when the selected frame does not have a trace
      function
    

    The attached patch fixes all those problems (by first removing the
    changes made in bdb.py at bpo-13183).

    After the patch, the implementation ensures now that self.stopframe is
    either None, or belongs to the stack frame in the interval
    [self.botframe, self._curframe] and that it is set to self.botframe
    when the debugging session terminates. This allows removing the while
    loop in stop_here with an improvement in the performance of Pdb (since
    stop_here may be called at each line, even when no breakpoint is set
    in the function).

    The patch applies to the default branch and includes 5 new test cases.
    A patch for 2.7 will be submitted later.

    @xdegaye xdegaye mannequin added stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error labels May 5, 2012
    @xdegaye
    Copy link
    Mannequin Author

    xdegaye mannequin commented May 5, 2012

    Uploaded pdb_default.patch that applies on the default branch with
    minor changes to the initial pdb.patch.

    @xdegaye
    Copy link
    Mannequin Author

    xdegaye mannequin commented May 5, 2012

    Uploaded pdb_2.7.patch that applies on the 2.7 branch.

    @terryjreedy
    Copy link
    Member

    Since this pdb issue is a continuation of bpo-13183 and repeals a part of that issue's patch, I nosy'ed the contributors to that issue.

    @xdegaye
    Copy link
    Mannequin Author

    xdegaye mannequin commented Nov 24, 2012

    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    @gaogaotiantian gaogaotiantian self-assigned this May 12, 2024
    miss-islington pushed a commit to miss-islington/cpython that referenced this issue May 13, 2024
    …e is not set (pythonGH-118979)
    
    (cherry picked from commit f526314)
    
    Co-authored-by: Tian Gao <gaogaotiantian@hotmail.com>
    iritkatriel pushed a commit that referenced this issue May 13, 2024
    …ce is not set (GH-118979) (#119007)
    
    gh-58933: Make pdb return to caller frame correctly when f_trace is not set (GH-118979)
    (cherry picked from commit f526314)
    
    Co-authored-by: Tian Gao <gaogaotiantian@hotmail.com>
    iritkatriel pushed a commit that referenced this issue May 13, 2024
    …ce is not set (GH-118979) (#119008)
    
    * [3.12] gh-58933: Make pdb return to caller frame correctly when f_trace is not set (GH-118979)
    (cherry picked from commit f526314)
    
    Co-authored-by: Tian Gao <gaogaotiantian@hotmail.com>
    @hugovk hugovk closed this as completed Jun 15, 2024
    estyxx pushed a commit to estyxx/cpython that referenced this issue Jul 17, 2024
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error
    Projects
    None yet
    Development

    No branches or pull requests

    3 participants