moving transport handler code to functions (MirrorNetworking#2444)

this matches Networkclient

Minor code cleanup in PlayerControllers

feat: Provide user friendly error with undefined reader/writer (Mirro…

…rNetworking#2423)

Previously this just gave NRE, which does not help the user address the problem.

provides a better error for MirrorNetworking#2411

breaking: Force networkmessage as struct everywhere. Redundant Messag…

…ePacker.GetId(type) not needed anymore (MirrorNetworking#2413)

* breaking: Force NetworkMessage as struct everywhere

* Redundant GetId(type) is not needed anymore

fix(SimpleWebTransport): Error event should now correctly be called

feat: Change NetworkPingDisplay settings at runtime (MirrorNetworking…

…#2396)

* making fields public
* adding format for text
* updating style each frame to update when it is changed

fix: flaky Weaver tests. Read/WritePacked removed for good

fix(MiddlewareTransport): making sure MiddlewareTransport listens for…

… inner events (MirrorNetworking#2386)

* making sure events are called

* full tests for middleware Transport

fix: Examples No Longer Rely On Sequential connectionId (MirrorNetwor…

…king#2387)

* NetworkRoomManager

* Basic Example

* Room Example

* MultipleAdditiveScenes Example

* Removed unused usings

* Removed redundant initializer

* Removed redundant initializer

* Renamed playerNo to playerNumber

Co-authored-by: MrGadget1024 <chris@clevertech.net>

fix: fixing first import of package (MirrorNetworking#2384)

* removing #if MIRROR

* removing #if MIRROR_26_0_OR_NEWER

* Update KcpTransport.cs

Co-authored-by: vis2k <info@noobtuts.com>