suzanne.soy/racket
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cs: more clearing of a terminated thread

Browse Source 
In particular, clear out the `suspend+resume-callbacks` field so that
retaining the thread object does not retain those callbacks.
This commit is contained in:
Matthew Flatt 2020-02-01 07:58:25 -07:00
parent 67595cc255
commit c81228f31e
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
racket/src/thread/thread.rkt
View File

@ -129,9 +129,7 @@
[cpu-time #:mutable] ; accumulates CPU time in milliseconds
[future #:mutable] ; current would-be future
[condition-wakeup #:mutable])
[future #:mutable]) ; current would-be future
#:authentic
#:property host:prop:unsafe-authentic-override #t ; allow evt chaperone
#:property prop:waiter
@ -203,8 +201,6 @@
0 ; cpu-time
#f ; future
void ; condition-wakeup
))
((atomically
(define cref (and c (custodian-register-thread c t remove-thread-custodian)))
@ -272,11 +268,14 @@
(thread-unscheduled-for-work-tracking! t))
(remove-from-sleeping-threads! t)
(run-kill-callbacks! t)
(set-thread-suspend+resume-callbacks! t null)
(when (thread-forward-break-to t)
(do-break-thread (thread-forward-break-to t) 'break #f))
(for ([cr (in-list (thread-custodian-references t))])
(unsafe-custodian-unregister t cr))
(set-thread-custodian-references! t null))
(set-thread-custodian-references! t null)
(set-thread-mailbox! t #f)
(set-thread-mailbox-wakeup! t void))
;; ----------------------------------------
;; Thread termination