Major Netcode Refactor

[valkyrienyanko]

Notes

The reflection code in PacketRegistry will be kept as it completely removes the need to manually register each individual packet class. Maybe in the future a source generator could handle this task so the reflection cost on startup is completely removed.

Changelog

Performance

• Reflection code to lookup properties via NetSend attribute are cached by type rather than instance. In other words property reflection code was ran every time a new instance of the same packet was created. Imagine if 1000 position packets were sent, that's 1000 calls with reflection. Now since properties are cached by type the reflection only happens once per type.
• Packet types are cached in their constructors so GetOpcode() does not need to retrieve them on every lookup
• Assembly.GetExecutingAssembly().GetTypes() in PacketRegistry is now cached for single lookup
• MapPackets<T> now uses an array instead of a List
• IgnoredPackets now uses a HashSet instead of List

Readability

• All packet registration code has been moved to PacketRegistry
• Godot netcode has been separated from ENet netcode by adding new classes GodotServer GodotClient
• All of the coding guidelines have been enforced

Usability

• An example scene has been provided in res://Framework/GodotUtils/Netcode/Sandbox/TopDown/
• Client and server factory classes are no longer required to be created for setup code
• NetControlPanelLow now requires its extending class to implement ENetOptions Options and all the UI button nodes
• All netcode has been wrapped with NETCODE_ENABLED which can easily be disabled by removing the constant from .csproj

Bug Fixes

• The server and client are now properly stopped when the main menu button is pressed in the game popup menu
• Server port is no longer fixed to default port and is fetched from the front end ui
• Fixed ToFormattedString(this object v) in GodotUtils returning extra blank lines
• Packets are now disposed of even when a System.IO.EndOfStreamException is thrown
• Server and client host will now be disposed of even if an exception is thrown
• Logger.Update() now only runs if NETCODE_ENABLED constant exists
• Added missing DEBUG define wrapped around Visualize using causing non-debug release builds to fail
• ValdiateResourceFile(...) in OptionsManager now also detects resource files with empty script paths caused by projects being ran in exported release mode