Skip to content
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

Message class overhaul #2

Open
dnut opened this issue Mar 17, 2019 · 0 comments
Open

Message class overhaul #2

dnut opened this issue Mar 17, 2019 · 0 comments

Comments

@dnut
Copy link
Owner

dnut commented Mar 17, 2019

This class was written for a specific purpose but doesn't generalize well. Source, destination, and payload would probably be better than typ, value, state, and source. Type basically refers to destination and value+state is sort of payload that is not generic enough. Payload structure should be encapsulated into its own class depending on the usage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant