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

One liner to fix debugger printing stack traces when lines of context are larger than source. #2717

Merged
merged 2 commits into from Dec 23, 2012

Conversation

bfroehle
Copy link
Contributor

Correctly print stack traces when context is larger than lines of source.

To test:

In [6]: def bar():
   ...:     return 20
   ...:

Before:

In [7]: ipdb.runcall(bar)
 <ipython-input-6-d0ed3bd5e16c>(2)bar()
          0     return 20

And here's after:

In [6]: ipdb.runcall(bar)
 <ipython-input-5-d0ed3bd5e16c>(2)bar()
          1 def bar():
    ----> 2     return 20

Replaces gh-2697. (Adds a test.)

tcmulcahy and others added 2 commits December 17, 2012 10:19
Correctly print stack traces when context is larger than lines of source.
bfroehle added a commit that referenced this pull request Dec 23, 2012
One liner to fix debugger printing stack traces when lines of context are larger than source.
@bfroehle bfroehle merged commit 1f24243 into ipython:master Dec 23, 2012
minrk added a commit that referenced this pull request Mar 5, 2013
…en lines of context are larger than source.

Correctly print stack traces when context is larger than lines of source.

To test:

```
In [6]: def bar():
   ...:     return 20
   ...:
```

Before:

```
In [7]: ipdb.runcall(bar)
 <ipython-input-6-d0ed3bd5e16c>(2)bar()
          0     return 20
```

And here's after:

```
In [6]: ipdb.runcall(bar)
 <ipython-input-5-d0ed3bd5e16c>(2)bar()
          1 def bar():
    ----> 2     return 20
```

Replaces gh-2697.  (Adds a test.)
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
One liner to fix debugger printing stack traces when lines of context are larger than source.
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