Various minor issues

[axcore]

A few minor code issues, none of which affect the user:

mobkit.get_box_displace_cols(), line 397-398

	local zsgn = sign(vec.z)
	local xsgn = sign(xoff)

The local is not necessary here, as the variables were declared above (or, rename these variables.)

mobkit.hq_aqua_attack()

"local tyaw" is defined twice, lines 781 and 811
"local yaw" is defined twice, lines 793 and 813

aqua_radar_dumb()

"local ok,h" appears twice, lines 693 and 702

mobkit.physics()

On line 688, "colinfo" is used without being declared, therefore it's global. Adding local colinfo = self.colinfo above will fix the problem.

mobkit.get_next_waypoint_fast()

Line 169, "liq" has not been declared (but "height" and "pos2" have)

[axcore]

There are also a couple of deprecated functions in use: :get_player_velocity() should be switched for :get_velocity(), and .initial_properties should be used in entity definitions.

[The4codeblocks]

There are also a couple of deprecated functions in use: :get_player_velocity() should be switched for :get_velocity(), and .initial_properties should be used in entity definitions.

mobkit intends to support older versions, are you sure?

[LuantistDork]

I don't think getting rid of some harmless warnings in the debug log is worth sacrificing backwards compatibility, best to keep :get_player_velocity() unless it's completely dropped in a future Luanti version.