Explain the difference between fixupCorridor and dtMergeCorridorStartMoved functions

[elsid]

The fixupCorridor function is defined here as a part of RecastDemo tool and dtMergeCorridorStartMoved is a part of DetourCrowd. The functions are identical from semantic point of view. The only difference in code is use of rcMin vs dtMin and rcMax vs dtMax functions. Why not use dtMergeCorridorStartMoved instead of fixupCorridor and remove fixupCorridor to avoid code duplication?

[grahamboree]

Thanks for pointing this out! It's likely that this was an oversight and RecastDemo should be using dtMergeCorridorStartMoved rather than re-implementing it. I've added this as an issue: #634

For context, we try to make the coupling of recast and detour as minimal as possible. The idea is that you should (in theory at least) be able to use either independent of the other. As a result, there are a handful of helper functions that are duplicated in both recast and detour and only differ by their rc or dt prefix.