actor-framework / actor-framework/incubator

Rename Transport/Application API

Aperta
#80 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
refactoring
Lingua principale
C++
Stelle
7
Fork
2
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

We should think about using more general naming for the transport and application protocol APIs. The Application layer is using a message-oriented communication model while the transport layer is using all different types depending on the transport protocol in use.
Hence, it would be more clear to rename `write_packet` in the transport abstraction to something more general such as `send` with some clear documentation about the underlying model (`UDP`->packet, `SCTP`->message, `TCP`->stream).
The same goes for `handle_data` in the application abstraction, which could be renamed to something along the lines of `receive`.

Do you have an opinion on that @Neverlord @josephnoir ?

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.