search

plasmodic / ecto

ecto is a dynamically configurable Directed Acyclic processing Graph (DAG) framework.
http://ecto.willowgarage.com/
BSD 3-Clause "New" or "Revised" License
97 stars 37 forks source link

Gil releaser #265

Closed stonier closed 9 years ago

stonier commented 9 years ago

This handles releasing of the gil from multiple threads.

The original was only saving one thread state globally and skipping release for any parallel threads. This one holds a single state for each thread.

I'm actually baffled why the original did it like this - it's a complete failure when calling the original releaser from a parallel thread. Any thoughts @vrabaud?

stonier commented 9 years ago

All tests passing...bugfixed the gil exercise as well.

vrabaud commented 9 years ago

lgtm