[webkit-reviews] review denied: [Bug 22614] Need to add Win32 implementation of ThreadSpecific. : [Attachment 25731] Proposed Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Dec 3 23:16:11 PST 2008

Alexey Proskuryakov <ap at webkit.org> has denied Jian Li <jianli at chromium.org>'s
request for review:
Bug 22614: Need to add Win32 implementation of ThreadSpecific.

Attachment 25731: Proposed Patch

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
Sorry, I pasted a wrong MSDN link, it was supposed to be

MSDN suggests calling TlsFree from DLL_PROCESS_DETACH, and it is indeed OK to
call it manually, or even not to call it at all - but I'm not talking about
leaking TLS keys. The problem is about leaking TLS data of each thread. In
pthread implementation of this class, destroy() is called automatically for
each ThreadSpecific object on thread exit, because the callback is registered
by pthread_key_create() - but nothing calls destroy() in this patch.

More information about the webkit-reviews mailing list