diff options
author | Michael W. Hudson <mwh@python.net> | 2002-11-08 13:08:46 (GMT) |
---|---|---|
committer | Michael W. Hudson <mwh@python.net> | 2002-11-08 13:08:46 (GMT) |
commit | 006c75265fe815fefadd119d6c7f322b1105c9bb (patch) | |
tree | b68ed1d8d49737841c6e1a42a1408a7e0934a9e0 /Python/ceval.c | |
parent | 019a78e76d3542d4d56a08015e6980f8c8aeaba1 (diff) | |
download | cpython-006c75265fe815fefadd119d6c7f322b1105c9bb.zip cpython-006c75265fe815fefadd119d6c7f322b1105c9bb.tar.gz cpython-006c75265fe815fefadd119d6c7f322b1105c9bb.tar.bz2 |
This is Richie Hindle's patch:
[ 631276 ] Exceptions raised by line trace function
It conflicted with the patches from Armin I just checked it, so I had
to so some bits by hand.
Diffstat (limited to 'Python/ceval.c')
-rw-r--r-- | Python/ceval.c | 23 |
1 files changed, 16 insertions, 7 deletions
diff --git a/Python/ceval.c b/Python/ceval.c index 1c95cb9..e654efd 100644 --- a/Python/ceval.c +++ b/Python/ceval.c @@ -51,7 +51,7 @@ static int call_trace(Py_tracefunc, PyObject *, PyFrameObject *, static void call_trace_protected(Py_tracefunc, PyObject *, PyFrameObject *, int); static void call_exc_trace(Py_tracefunc, PyObject *, PyFrameObject *); -static void maybe_call_line_trace(Py_tracefunc, PyObject *, +static int maybe_call_line_trace(Py_tracefunc, PyObject *, PyFrameObject *, int *, int *); static PyObject *apply_slice(PyObject *, PyObject *, PyObject *); @@ -726,9 +726,14 @@ eval_frame(PyFrameObject *f) /* see maybe_call_line_trace for expository comments */ f->f_stacktop = stack_pointer; - maybe_call_line_trace(tstate->c_tracefunc, - tstate->c_traceobj, - f, &instr_lb, &instr_ub); + + if (maybe_call_line_trace(tstate->c_tracefunc, + tstate->c_traceobj, + f, &instr_lb, &instr_ub)) { + /* trace function raised an exception */ + why = WHY_EXCEPTION; + goto on_error; + } /* Reload possibly changed frame fields */ JUMPTO(f->f_lasti); stack_pointer = f->f_stacktop; @@ -2872,7 +2877,7 @@ call_trace(Py_tracefunc func, PyObject *obj, PyFrameObject *frame, return result; } -static void +static int maybe_call_line_trace(Py_tracefunc func, PyObject *obj, PyFrameObject *frame, int *instr_lb, int *instr_ub) { @@ -2947,6 +2952,8 @@ maybe_call_line_trace(Py_tracefunc func, PyObject *obj, suggested by f_lasti on this one occasion where it's desirable. */ + int result = 0; + if ((frame->f_lasti < *instr_lb || frame->f_lasti >= *instr_ub)) { PyCodeObject* co = frame->f_code; int size, addr, line; @@ -2980,8 +2987,8 @@ maybe_call_line_trace(Py_tracefunc func, PyObject *obj, if (addr == frame->f_lasti) { frame->f_lineno = line; - call_trace(func, obj, frame, - PyTrace_LINE, Py_None); + result = call_trace(func, obj, frame, + PyTrace_LINE, Py_None); } if (size > 0) { @@ -2996,6 +3003,8 @@ maybe_call_line_trace(Py_tracefunc func, PyObject *obj, *instr_ub = INT_MAX; } } + + return result; } void |