-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Event Based crap #1
Comments
How about this https://gist.github.com/Blackrush/7345399 but users have to pass an |
C'est une implem. details qui leak :(. |
Et j'aime bien l'idée de check que c'est un event qui peut être emit même si sans compile time check c'est pas super utile |
Où ça ça leak ? Pour le check des events faudrait les définir avant d'emit |
implicit val executor = Executors.newSingleThreadExecutor() la classe/l'objet devraient pas avoir à s'occuper de ça |
Ouais, d'où mon commentaire plus haut mais je doute qu'un seul Executor pour toute la VM fasse l'affaire |
Mettre sa création dans le constructeur d'Event ? |
Un Thread par Observable ? Bof bof |
:(. même faire un pool par type serait juste faire le devin ... |
D'un autre côté, est-ce que c'est si important qu' |
Your idea ;) |
Oui j'sais bien :D mais j'voulais savoir ton avis |
non je pense pas que ça soit très important, et au pire on peut faire un overload AsyncObservable plus tard si le besoin s'en fait sentir. |
Voilà c'est fait, j'ai ajouté une méthode 'emitted' pour marquer les events potentiellement déclenchés. J'commit des que j'ai un accès internet. |
k nice :). |
Look b2c7a7c j'aime pas du tout |
Pourquoi t'en as vraiment besoin ? Pourquoi |
|
oui j'ai bien répondu après :(. |
Tight event-loop.
The text was updated successfully, but these errors were encountered: