search

tonarino / actor

A minimalist actor framework aiming for high performance and simplicity.
MIT License
40 stars 6 forks source link

First version of the system spawn builder pattern #41

Closed bschwind closed 3 years ago

bschwind commented 3 years ago

Closes #10

I need to add more documentation but otherwise should be ready for review.

bschwind commented 3 years ago

@strohel @mcginty @skywhale I changed the blocking function to be named run, as block_on is a bit weird when you don't pass it any arguments. Do you have any other suggestions for the name?

mcginty commented 3 years ago

@strohel @mcginty @skywhale I changed the blocking function to be named run, as block_on is a bit weird when you don't pass it any arguments. Do you have any other suggestions for the name?

My only beef is that run and spawn are almost synonymous. run_and_block might be more understandable even if it's a bit verbose?

bschwind commented 3 years ago

run_and_block, blocking_run - are there any other suggestions? I'll wait a bit on this and see what the consensus is.

strohel commented 3 years ago

Yeah run_and_block() is cool, it's true that run_on_main() was misleading.