cybertf2k4
Active Member
I was looking through eq2bot.iss and noticed what might be a bug in the code for MoveToMaster function...
There seems to be a missing pair of "}]'s" in the "Move to next Waypoint" section
the original lines read as follows...
based on these code lines, I would assume this function should throw an error or at least provide incorrect values because the Math.Calc is not closed properly. Maybe innerspace takes care of this on its own but I wanted to make sure that as I am learning, I am also understanding.
I think these lines should be corrected to the following...
This should allow the Math.Calc to complete and correctly set WPX and WPZ
Please take a look and let me know if I am correct or off base here. I cannot test this now as I am at work.
CyberTF2k4
There seems to be a missing pair of "}]'s" in the "Move to next Waypoint" section
the original lines read as follows...
Code:
WPX:Set[${Math.Calc[${tmpdev}*${Math.Cos[-${Me.Heading}]}+${NavPath.Point[${pathindex}].X}]
WPZ:Set[${Math.Calc[${tmpdev}*${Math.Sin[-${Me.Heading}]}+${NavPath.Point[${pathindex}].Z}]I think these lines should be corrected to the following...
Code:
WPX:Set[${Math.Calc[${tmpdev}*${Math.Cos[-${Me.Heading}]}+${NavPath.Point[${pathindex}].X}]}]
WPZ:Set[${Math.Calc[${tmpdev}*${Math.Sin[-${Me.Heading}]}+${NavPath.Point[${pathindex}].Z}]}]Please take a look and let me know if I am correct or off base here. I cannot test this now as I am at work.
CyberTF2k4
Last edited: