Duplicately named and unsafe to upvalue moho functions

[ChessBerry]

Some moho functions in different files have the same name. E.g.
CSlideManipulator:SetSpeed(speed)
and
CRotateManipulator:SetSpeed(self, degrees_per_second)
are both "SetSpeed" functions. In the above case you can maybe distinguish between them by the number of arguments, but there are quite a few more of them.

From the sim part of the moho functions the duplicates are:
{'CanAttackTarget', 'GetFactionIndex', 'IsOpponentAIRunning', 'AddAgentFalloffForce', 'GetBlueprint', 'SetResourceCap', 'Stop', 'base', 'GetSkirmishBase', 'IsMoving', 'SetHeadingPitch', 'GetCurrentTargetPos', 'SetEnabled', 'GetLauncher', 'GetVelocity', 'PlaySound', 'IsEnabled', 'RemoveBuildRestriction', 'GetMaxHealth', 'CreateProjectile', 'SetSpeed', 'Destroy', 'GetPersonality', 'AddBuildRestriction', 'GetWeaponCount', 'SetAcceleration', 'AddResourceManagerType', 'SetRegenRate', 'print', 'GetHealth', 'SetGoal', 'GetPosition', 'Enable'}

"base" and "GetBlueprint" appear quite a few times, both of which we don't want to touch anyway though.

[ChessBerry]

Seemingly, upvaluing
local UnitWeaponMethods = _G.moho.weapon_methods
local UnitWeaponMethodsChangeMaxRadius = UnitWeaponMethods.ChangeMaxRadius

also leads to a weird error when trying it to upvalue it in the cybran commander file URL0001_script.lua
Idk, I just removed the lines

    "ChangeMaxRadius": "INFO: UnitWeapon:ChangeMaxRadius(value)",
    "ChangeMinRadius": "INFO: UnitWeapon:ChangeMinRadius(value)",

from the moho sim reference file for now

[ChessBerry]

Removed "Enable" and "Disable" functions from the moho reference in 962a562, as similarly named lua functions were upvalued incorrectly to those

[ChessBerry]

Further removed the following potentially unsafe to upvalue moho references in commit dae4d40:

"Sim.<global>"
        "Damage": "INFO: Damage(instigator, origin, target, amount, damageType)",
        "GetBlueprint": "INFO: blueprint = GetBlueprint(entity)",
         "print": "INFO: Print a log message",

"Sim.AiSkirmishBase": {
        "print": "INFO: Print a log message",

"Sim.CAiAttackerImpl": {
        "Stop": "INFO: Stop the attacker",

"Sim.CPlatoon": {
        "Destroy": "INFO: CPlatoon:Destroy()",
        "Stop": "INFO: CPlatoon:Stop()",

"Sim.CollisionBeamEntity": {
        "IsEnabled": "INFO: bool = CollisionBeamEntity:IsEnabled()",

"Sim.Entity": {
        "GetBlueprint": "INFO: blueprint = Entity:GetBlueprint()",
        "Destroy": "INFO: Entity:Destroy()",
        "Kill": "INFO: Entity:Kill(instigator,type,excessDamageRatio)",
        "PlaySound": "INFO: Entity:PlaySound(eventname,cull)",

"Sim.UnitWeapon": {
        "GetBlueprint": "INFO: blueprint = UnitWeapon:GetBlueprint()",
        "PlaySound": "INFO: UnitWeapon:PlaySound(weapon,eventName)",

I don't know if all of them actually are unsafe, but as the editor doesn't understand the code and instead just upvalues all lua functions of the same name, having moho functions with a very generic name is almost certainly a bad idea.
See e.g. the problems that the "Disable" upvalue above caused by incorrectly upvaluing other lua functions called "disable" to the moho fct "disable" even though they had nothing much in common, besides the name.
Also, some of them were duplicately named and hence weren't upvalued with the current editor code anyway.

[ChessBerry]

This upvalue:

self:GetWeaponByLabel('Suicide'):FireWeapon()

to

self:GetWeaponByLabel('Suicide')
UnitWeaponMethodsFireWeapon(self)

makes problems in the firebeetle file, the only place where it is upvalued, which probably means that the Fireweapon upvalue is unsafe. Removed it in 2222ce0

[ChessBerry]

Similarly to the above, the upvalue of SetCollision in SBOOhwalliStrategicBomb01_script.lua causes a desync and was hence removed from the moho reference in e5f89ba