search

PolyProcessInterface / ppi

2 stars 2 forks source link

infra.wait() et infra.notifayAll() #24

Closed n-peugnet closed 4 years ago

n-peugnet commented 4 years ago

Probablement d'abord déplacer le code qui lance un nouveau thread depuis NodeProcess.handleMessage() vers Infrastructure en ajoutant une fonction intermédiaire.

max0218 commented 4 years ago

Pourquoi vous vouliez pas garder le message handler et le multithreading dans NodeProcess et y ajouter le wait et notifayAll ?

n-peugnet commented 4 years ago

hmm parce que je trouvais ça plus clair pour l'api qu'on est en train de créer que tous les appels se fassent à infra. Mais c'est vrai que c'est pas forcément plus logique de faire infra.wait() que de faire this.wait(). étant donnée que c'est justement le process qui wait et non l'infra...

Bon je suis partagé. On peut peut-être le laisser comme ça au final oui.