refactor(pathfinder): Remove unused function pathDestination

[Caball009]

Function Pathfinder::pathDestination isn't used now nor in the first usable branch, so this PR removes it.

Fortunately, it wasn't used because there would likely be a bug due to the uninitialized coordinate variable here, same as Pathfinder::tightenPathCallback:

GeneralsGameCode/GeneralsMD/Code/GameEngine/Source/GameLogic/AI/AIPathfind.cpp

Line 8196 in 5d9aa65

Coord3D pos;

[greptile-apps]

Greptile Overview

Greptile Summary

Removed the unused Pathfinder::pathDestination function from both Generals and GeneralsMD codebases. This function was never called and contained a potential uninitialized variable bug similar to tightenPathCallback.

• Removed function declaration from header files (4 lines total across both versions)
• Removed 251-line function implementation from source files (502 lines total across both versions)
• Verified no references to this function exist in the codebase

Confidence Score: 5/5

• This PR is safe to merge with no risk
• Removes dead code with no references in the codebase, preventing potential bugs from unused code paths
• No files require special attention

Important Files Changed

Filename	Overview
Generals/Code/GameEngine/Include/GameLogic/AIPathfind.h	Removed unused pathDestination function declaration
Generals/Code/GameEngine/Source/GameLogic/AI/AIPathfind.cpp	Removed unused 251-line pathDestination function implementation
GeneralsMD/Code/GameEngine/Include/GameLogic/AIPathfind.h	Removed unused pathDestination function declaration
GeneralsMD/Code/GameEngine/Source/GameLogic/AI/AIPathfind.cpp	Removed unused 251-line pathDestination function implementation

_{Last reviewed commit: 50dc090}

[Mauller]

I will double check this again, the code appears unused, i believe it was an earlier implementation of what ended up split out into multiple functions.