5154 2005-09-27 11:54:49 -0700 JSC should switch from Core Foundation to unix time/date functions 2005-10-09 14:08:54 -0700 1 1 1 Unclassified WebKit JavaScriptCore 420+ Mac OS X 10.4 RESOLVED DUPLICATE 5280 P2 Normal --- 1 ggaren darin ap darin oldest_to_newest 20672 0 ggaren 2005-09-27 11:54:49 -0700 There are two parts to this task: 1. Change e.g. localtime calls to e.g. localtime_r calls. 2. Remove Core Foundation calls and replace with true unix calls. The CF stuff was a performance hack that we no longer need. The non-_r time/date functions are not thread-safe or re-entrant. This can cause surprising bugs due to more than one time/date call in the same call stack or using date code on multiple threads. 20675 1 ap 2005-09-27 13:36:43 -0700 Actually, on Darwin, localtime() uses a per-thread global <http://darwinsource.opendarwin.org/10.4.2/ Libc-391/stdtime/FreeBSD/localtime.c>. A lot of other functions that used to operate on static globals do the same. MS Windows doesn't have localtime_r at all (if Windows portability is a concern). 20981 2 ggaren 2005-09-30 08:53:48 -0700 OK, thread safety is no longer a concern. The call stack problem remains, but I suppose that just calls for careful coding. So, let's use this bug just to track the switch away from CF calls. 21601 3 darin 2005-10-08 23:24:55 -0700 To fix bug 5280, I recommended we do this. And my patch attached to bug 5280 does this. 21626 4 ggaren 2005-10-09 14:08:54 -0700 Changing the bug name to reflect that the remaining work is to switch away from CF calls. *** This bug has been marked as a duplicate of 5280 ***