actor-framework / actor-framework/incubator

Rename Transport/Application API

Open
#80 0 comments 0 reactions 0 assignees View on GitHub
refactoring
Dominant language
C++
Stars
7
Forks
2
PR merge metrics
No merged PRs in 30d

Description

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 ?

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.