On Sun, May 10, 2015 at 6:59 AM, Nick Wellnhofer <[email protected]> wrote:
> I ported t/binding/038-lock_free_registry.t to C to really finish this off: > > https://github.com/nwellnhof/lucy-clownfish/commits/port_lfreg_test_to_c It will be great to have this test enabled again! The branch looks good to me in general. I have two notes. One is that `pthread_yield` is not portable. I tried to run this branch on my OS X laptop and it wouldn't compile. The second is that the concurrency abstraction may not map to every host elegantly. For example, under Go, running goroutines won't give us thread-local storage to hold Err_error. But we can deal with each host case-by-case and disable the test if necessary when the time comes. Dunno how hard it will be to work around pthread_yield, but +1 to merge everything else I see here. Marvin Humphrey
