document function BotAimAtLocation

[sweet235]

This is just comments. Document the slightly unexpected behavior of the function BotAimAtLocation.

I would propose a concrete solution, but I am still learning how to handle the angles pitch, yaw, roll.

[sweet235]

Well, I am not sure if I understand what is going here. This function seems to work with absolute coordinates iff the bot is perfectly horizontal. I will try to understand this before I make an attempt to merge this.

[sweet235]

Turns out someone™ exploited this behavior in BotClimbToGoal in sg_bot_nav.cpp:

static void BotClimbToGoal( gentity_t *self )
{
	glm::vec3 ownPos = VEC2GLM( self->s.origin );
	glm::vec3 dir = self->botMind->nav().glm_dir();
	BotAimAtLocation( self, ownPos + 100.f * dir );     // this makes the bot look straight ahead on walls/ceilings, not sure why
	self->botMind->cmdBuffer.upmove = -127;
	self->botMind->cmdBuffer.forwardmove = 127;
	self->botMind->cmdBuffer.rightmove = 0;
}

I remember wondering why this works on walls and ceilings. Maybe the current behavior is, in a way, reasonable.

[slipher]

I don't get what is the unexpected part.

	glm::vec3 ownPos = VEC2GLM( self->s.origin );
	glm::vec3 dir = self->botMind->nav().glm_dir();
	BotAimAtLocation( self, ownPos + 100.f * dir );

Wouldn't this always be expected to make the bot look straight ahead?