From: Jan N. <nij...@us...> - 2013-08-01 12:55:50
|
2013/7/26 Donald G Porter <don...@ni...>: > On 07/25/2013 03:13 AM, Jan Nijtmans wrote: >> I would like to merge this to core-8-5-branch. Any objections? Anyone who >> would like to test it on Solaris 10 and/or OSX? > > Test passes on Solaris 10, threaded and not. Actually, I didn't expect otherwise, but still: Thanks! Merged to core-8-5-branch and trunk now. This doesn't mean that there is no room for further improvement! In this commit: <http://core.tcl.tk/tcl/info/ec1a5f8686> Daniel Steffen modified the OSX notifier, creating the notifier thread in Tcl_WaitForEvent() in stead of in Tcl_InitNofier(). That would mean that in Rivet on OSX the notifier thread doesn't exist yet when the fork() runs, which prevents the whole problem we are handling here. That would be a good improvement for the Unix Notifier as well. Regards, Jan Nijtmans |