syncInterval resets back to 0.1f (Value that is set in the NetworkBehaviour class) if changed using the editor. The issue is present when changing both, scene objects as well as prefabs.
* fix
* Test file
* Syntax
* NetworkBehaviour.Set/GetSyncVarNetworkIdentity functions added
* add definitions to weaver
* Use them
* add NetworkIdentity cases everywhere else too
* ProcessInstructionSetter/GetterField replaces for OnDeserialize too, so that [SyncVar]GameObjects/NetworkIdentities work with custom On(De)Serialize as well
* Custom on(De)serialize test
* Weaver.ProcessSiteMethod doesn't ignore OnDeserialize anymore either
* [SyncVar] GameObject/NetworkIdentity hooks work again
* testscene
* public gameobject field test
* pass gameobject field to getsyncvargameobject etc.
* GetSyncVarGameObject/NetworkIdentity always returns the field if server, looks up netId otherwise; functions aren't static anymore.
* Use original OnSerialize generation again, so it simply calls writer.Write(go) on server, which sends the netId internally anyway.
* fixed hooks not using 'this.' because getter function isn't static anymore.
* add scene referencing test
* remove tests
* Fix additive scene loading causing duplicate sceneIds by using an offset per scene.
* TEST: moved SpawnableObjects to NetworkIdentity. OnDisable adds it to the dict. SpawnObject removes it.
* Log if we don't find the spawn scene object
* Fix compilation issues
* Make PrepareToSpawnSceneObjects public
* 2018.3 warning fix. The old way to check if something is a porefab is depricated so needs replacing with new code.
* 2018.3 warning fix. The old way to check if something is a porefab is depricated so needs replacing with new code.
* Bug fix for 2018.3+ where NetworkIdentity GUIDs are set to blank in prefab files and shown as blank in the editor.
* Another 2018.3 warning fix. The old way to check if something is a porefab is depricated so needs replacing with new code.
* Make sure it works in 2017.4
* Change not needed
* Revert "Change not needed"
This reverts commit 13747739df.
* Mark LLAPI as obsolete
* Don't use obsolete api
* Suppress obsolete LLAPI warning for now
* Remove warning
* Make conditional a little clearer
* Ignore unity generated files and folders
* Import mirror as source
* Source based does not build dlls anymore
* nuget packages not used in source based
* Produce an archive with the release
* generate archive with Mirror folder to avoid accidents
* Deploy to github releases
* Document new installation procedures
The RemovePlayerController call sets the playerController to null, causing a null dereference (and a memory leak as pointed out by @paulpach, since gameobjects are never destroyed)
