Revisit Packet implementation - Githubissues

ChaseHuegel / swordfish-engine

C# engine built on ECS and multithreading with high performance and modding in mind. Targeting PBR-based Low-Fi and 2.5D visuals with VR support!

Apache License 2.0

4 stars 0 forks source link

Revisit Packet implementation #57

Closed ChaseHuegel closed 1 year ago

ChaseHuegel commented 2 years ago

Cleanup the class
Reduce redundancy, implement more code reuse in reading
Remove explicit if-else statements
Broaden supported types
Investigate the value of introducing custom written compressed types
Fallback to object (de)serialization on unsupported types

ChaseHuegel commented 1 year ago

This was resolved in commit https://github.com/ChaseHuegel/swordfish-engine/commit/74b223c2b1e0a775e5c85244483eb1857f1257b1 after (de)serialization was migrated to the new Needlefish library.