diff options
| author | Ned Batchelder <ned@nedbatchelder.com> | 2017-03-03 06:56:21 -0500 |
|---|---|---|
| committer | Ned Batchelder <ned@nedbatchelder.com> | 2017-03-03 06:56:21 -0500 |
| commit | 68fb56eb803283f7df9caf2d47c0d39e7be95cbb (patch) | |
| tree | dd473a0ae57e68589de94e267a898b7d299e6226 /coverage/ctracer/tracer.h | |
| parent | 0c38af32a0cada8d35671adff86c871d3badb37a (diff) | |
| download | python-coveragepy-git-68fb56eb803283f7df9caf2d47c0d39e7be95cbb.tar.gz | |
Simplify stack management in CTracer
"file_data" used to be borrowed from data, but that was confusing. Now it's
owned. We used to have a struct member which was a copy of the current stack
entry. That just made it harder to reason about reference counting. Now we
have a pointer to the entry on the stack.
Diffstat (limited to 'coverage/ctracer/tracer.h')
| -rw-r--r-- | coverage/ctracer/tracer.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/coverage/ctracer/tracer.h b/coverage/ctracer/tracer.h index 438317b0..c174ae5a 100644 --- a/coverage/ctracer/tracer.h +++ b/coverage/ctracer/tracer.h @@ -54,8 +54,8 @@ typedef struct CTracer { int data_stacks_used; DataStack * pdata_stack; - /* The current file's data stack entry, copied from the stack. */ - DataStackEntry cur_entry; + /* The current file's data stack entry. */ + DataStackEntry * pcur_entry; /* The parent frame for the last exception event, to fix missing returns. */ PyFrameObject * last_exc_back; |