Coordinated spawns hotfixes

[Metious]

Changes made in this pull request

• Fixed a bug where the savedSpawns Deserializer was failing.
• Added two new public JsonConverter classes (QuaternionConverter/Vector3Converter) for simplified serializations.
• Redesigned SpawnInfo into a struct, implementing IEquatable<SpawnInfo>, custom Equals, GetHashCode methods and operator overloads for == and != respectively.
• Fixes an issue where EntitySpawner.SpawnAsync was using TechType.ToString() rather than TechType.AsString().

Builds by @tobeyStraitjacket

5555948 v2.10.0.3: Equals(object obj) & GetHashCode()

• This build features an implementation as initially intended by @Metious which overrides Equals(object obj) & GetHashCode() to provide a value-based equality comparison for all properties of SpawnInfo.
• As it overrides Equals and GetHashCode to provide value-based equality comparison, this is a breaking change for anyone who previously relied on Equals or GetHashCode (particularly a problem in the case of HashSet), but it solves the problem where List.Contains is returning false for deserialized SpawnInfo instances. However, it also means that we cannot later change the implementation of Equals or GetHashCode without another breaking change for consumers.

Subnautica

Stable v2.10.0.3
Experimental v2.10.0.3

Below Zero

Stable v2.10.0.3
Experimental v2.10.0.3

96cdaa3 v2.10.0.5: IEquatable<SpawnInfo> Equals(SpawnInfo other)

• This build does not override Equals(object obj) or GetHashCode, but instead implements IEquatable<SpawnInfo> and Equals(SpawnInfo other) to provide a value-based equality comparison for all properties of SpawnInfo.
• List.Contains should defer toEquals(SpawnInfo) for comparison since we implement IEquatable<SpawnInfo>, and the issue @Metious was trying to solve should be resolved.
• This should make for a less egregious backwards-incompatible change, as Equals(object obj) and GetHashCode still operate the same, it is only Equals(SpawnInfo other) that has been changed. Still, the issue remains that if we change this in future, it will have implications for consumers, so it must be approached with caution.

Subnautica

Stable v2.10.0.5
Experimental v2.10.0.5

Below Zero

Stable v2.10.0.5
Experimental v2.10.0.5

b9802ce v2.10.0.6: SpawnInfo redesigned as a struct

• This build combines the logic of both 2.10.0.3 and 2.10.0.5, and on top of that, also redesigns SpawnInfo as a struct rather than a class.
• @Metious assures me that nobody outside of himself and the RotA team are making any use of the SpawnInfo API, so a backwards-incompatible change that fixes the design of the API is preferable, should this prove to be true.

Subnautica

Stable v2.10.0.6
Experimental v2.10.0.6

Below Zero

Stable v2.10.0.6
Experimental v2.10.0.6

Builds by @Metious ca7018a

SMLHelper_BZ.STABLE.zip
SMLHelper_BZ.STABLE.zip

[Metious]

Calling for a fast merge/publish as this bug was game breaking.

[toebeann]

Have suggested changes in comments, I also intend to work on GetHashCode() implementation before this goes through as I believe there will be issues with SpawnInfo if we introduce an Equals method that is not inline with the GetHashCode method - particularly if someone ever wants to make a HashSet of SpawnInfo instances.
Simply deferring to base.GetHashCode() when you have modified the logic of Equals will get the compiler to "shut up", but it will also cause problems down the road.

[Metious]

Tested SMLHelper 2.10.0.6 and i can confirm that the potential bugs were fixed and everything works just fine.

[toebeann]

v2.10.0.6 appears to be bug free after testing from @Metious. Issues that were game-breaking are resolved, and no breaking changes for their 2.9.7 build of mods utilising the API.