Messages

[mikeyoung3k]

Merge IncomingMessage and OutgoingMessage into one Message struct.

This opens up the possibility of changing the NetService trait to just take a Message type instead of Payload and target.

TCP tests not currently running/working - a few other issues to fix before it can be readded.

I've not updated any call sites outside the net crate that use Incoming/OutgoingMessage yet - it looks like they all also require other work from architectural changes

[efimio16]

Thank you for the PR, here are a few things you could improve, and some plans changed a little bit:

1. Remove core/* from .gitignore
   Earlier in the project, before switching to the cargo workspace there was a folder core/, but now we don't need it. So just delete it from yourself

2. Let's delete Message
   Yes I know I said before that this there will be Frame - Packet - WireMessage - Message - Payload, but when I was planning next steps I realized that it would be better to replace it with Frame - Packet - WirePayload - Payload - PayloadBody because in this way we use only the "payload" word and it won't conflict with the Message from future crates

In general, thank you for supporting the project!

[mikeyoung3k]

Righto - will have another crack