Bug 1085 – Need to add ability to detach threads.

Status
RESOLVED
Resolution
FIXED
Severity
normal
Priority
P2
Component
druntime
Product
D
Version
D2
Platform
All
OS
All
Creation time
2007-03-28T15:40:18Z
Last change time
2021-05-19T04:52:29Z
Keywords
bootcamp
Assigned to
No Owner
Creator
Matt Brandt

Comments

Comment #0 by mattbd — 2007-03-28T15:40:18Z
It is not always reasonable to use wait() to clean up thread resources. I would like to see a detach method for threads or a DetachedThread class added to the library.
Comment #1 by sean — 2007-03-28T16:57:25Z
For what it's worth, Tango will implicitly detach any thread for which pthread_join has not been called by the time the the Thread object is destroyed. I think this is the preferred approach for D, since some required functionality of the Thread object may not work if the thread has been detached prematurely.
Comment #2 by braddr — 2007-03-28T20:32:35Z
This doesn't seem dgcc/gdc specific and assuming not should be re-assigned to the more general D / Phobos bucket.
Comment #3 by mattbd — 2007-03-28T23:20:32Z
I think that assuming that a thread should detach just because no thread called wait() before the thread exits is incorrect. It is certainly within the posix programming model to use wait to collect status of previously exited threads and that should probably be exposed. I think either an explicit detach() method, or a separate subclass for detached threads would be better than the Tango model. I agree that this is general to D and am moving it to that category.
Comment #4 by sean — 2007-03-29T01:05:24Z
[email protected] wrote: > > ------- Comment #3 from [email protected] 2007-03-28 23:20 ------- > I think that assuming that a thread should detach just because no thread called > wait() before the thread exits is incorrect. It is certainly within the posix > programming model to use wait to collect status of previously exited threads > and that should probably be exposed. I think either an explicit detach() > method This would be reasonable for Posix, but now Win32. On Posix, the thread handle isn't needed for any of the currently exposed functionality (though it may be for future functionality, such as modifying thread priorities). On Win32 however, the thread handle is required for pretty much everything, including suspending the thread for garbage collection. So the only recourse would be to make detach() a no-op on Win32, and to consider the use of some functions an error after detach() is called on Posix. > or a separate subclass for detached threads would be better than the > Tango model. The user is free to subclass Thread and do whatever he wants, subject to the aforementioned restrictions. But design issues at this level are better discussed elsewhere.
Comment #5 by code — 2014-05-27T18:55:27Z
*** Issue 12812 has been marked as a duplicate of this issue. ***
Comment #6 by razvan.nitu1305 — 2017-07-12T09:15:31Z
This is a druntime issue. Changing component to druntime
Comment #7 by edi33416 — 2019-05-28T15:20:42Z
Is this issue solved by core.thread : thread_detachByAddr, thread_detachInstance, thread_detachThis ? https://dlang.org/phobos/core_thread.html#.thread_detachByAddr
Comment #8 by edwards.ac — 2021-05-19T04:52:29Z
As indicated in comment #7, the issue is resolve by the features present in core.thread.threadbase.