[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

[roossienb]

One function that calls examineNeighboringCells() is internalFindPath(). In the latter there are multiple times that obj is being NULL tested. None of these are getting removed in this PR.

It feels a bit random to remove some, but not others.
Consider the spaghetti-mess pathfinding is and how hard it was to find the cause of crashes, I think this PR should not be merged.

[xezon]

As long as it does not cause mismatch, it should be fine. The game would have crashed if any of these null checks were really necessary, otherwise the compiler would not emit this warning.

[roossienb]

Could you rebase it to main , so I can mass test it?

[xezon]

Rebased to Main.