[GEN][ZH] Remove unnecessary NULL pointer tests in AIPathfind, AIPlayer, AIStates

[xezon]

This change removes unnecessary NULL pointer tests in AIPathfind, AIPlayer, AIStates and makes the compiler happy.

Unnecessary, because I saw no condition under which NULL would be passed to these pointers. They should always contain valid pointers, otherwise this would have crashed anyway.

GeneralsMD\Code\GameEngine\Source\GameLogic\AI\AIPathfind.cpp(6239): warning C6011: Dereferencing NULL pointer 'obj'.
GeneralsMD\Code\GameEngine\Source\GameLogic\AI\AIPathfind.cpp(6331): warning C6011: Dereferencing NULL pointer '<COMPLEX_EXPR>'. See line 6239 for an earlier location where this can occur
GeneralsMD\Code\GameEngine\Source\GameLogic\AI\AIPathfind.cpp(9386): warning C6011: Dereferencing NULL pointer 'objOther'.
GeneralsMD\Code\GameEngine\Source\GameLogic\AI\AIPathfind.cpp(10238): warning C6011: Dereferencing NULL pointer 'obj'.
GeneralsMD\Code\GameEngine\Source\GameLogic\AI\AIPathfind.cpp(10409): warning C6011: Dereferencing NULL pointer 'obj'.
GeneralsMD\Code\GameEngine\Source\GameLogic\AI\AIPathfind.cpp(10617): warning C6011: Dereferencing NULL pointer 'obj'.
GeneralsMD\Code\GameEngine\Source\GameLogic\AI\AIPathfind.cpp(10955): warning C6011: Dereferencing NULL pointer 'obj'.
GeneralsMD\Code\GameEngine\Source\GameLogic\AI\AIPlayer.cpp(2885): warning C6011: Dereferencing NULL pointer 'team'.
GeneralsMD\Code\GameEngine\Source\GameLogic\AI\AIStates.cpp(5237): warning C6011: Dereferencing NULL pointer 'm_att'. See line 5200 for an earlier location where this can occur