Clean up libasync driver module support

vibe-d / vibe.d

Official vibe.d development

MIT License

1.15k stars 284 forks source link

Clean up libasync driver module support #935

Open s-ludwig opened 9 years ago

s-ludwig commented 9 years ago

Follow up ticket for #814:

The module constructor workaround (querying the thread name) needs a closer look.
Public imports of libasync should be replaced by explicit conversions to vibe.d's types for the time being. Ultimately it would be good to use Phobos' types (with properly extended functionality) for both, vibe.d and libasync.

etcimon commented 9 years ago

Ultimately it would be good to use Phobos' types (with properly extended functionality) for both, vibe.d and libasync.

I've been holding this off because I'm working on a FreeListRef implementation of containers and a custom allocator red black tree, which will be used for the timers as well. Are you interested in adding the final size_t read(Vector!ubyte) to the streams? (where Vector uses a custom allocator). I think passing the allocator with the container would help minimize GC use