BaseGame Script Review

[lukaspj]

This issue was created in the GarageGames Repository (Link to original issue).
The issue was originally created by @lukaspj and had a total of 2 comments that may contain additional information. The original issue description is pasted below:

Just listing off some issues I encounter when converting scripts to C#:

Core_GameObjects module does not exist.
https://github.com/GarageGames/Torque3D/blob/development/Templates/BaseGame/game/core/Core.cs#L27

PrefPath logic exists twice
https://github.com/GarageGames/Torque3D/blob/development/Templates/BaseGame/game/core/Core.cs#L30-L34
And:
https://github.com/GarageGames/Torque3D/blob/development/Templates/BaseGame/game/main.cs.in#L34-L39

OnExit should not be a part of the Core Module, rather main.cs or similar root-level script should destroy the Core Module on exit
https://github.com/GarageGames/Torque3D/blob/development/Templates/BaseGame/game/core/Core.cs#L81-L87

I know this one is a bit subjective, but I think it's confusing to have CoreModule's clean-up logic in the root-level "OnExit" function.

Should TerrainBlendShader prefix samplerNames with $?, it seems to be prefixed everywhere else.
https://github.com/GarageGames/Torque3D/blob/development/Templates/BaseGame/game/core/rendering/scripts/gfxData/terrainBlock.cs#L32-L33

I'll add to this as I find more