Opened 7 years ago

Last modified 3 years ago

#4636 new defect

Territory Manager should exclude 3 territory tile border — at Version 1

Reported by: elexis Owned by:
Priority: Should Have Milestone: Backlog
Component: Simulation Keywords: simple
Cc: Patch:

Description (last modified by elexis)

In CCmpRangeManager.cpp we have

// WARNING: CCmpPathfinder::UpdateGrid needs to be kept in sync with this
const ssize_t edgeSize = 3; // number of vertexes around the edge that will be off-world

In CCmpPathfinder.cpp we have

// Compute off-world passability
// WARNING: CCmpRangeManager::LosIsOffWorld needs to be kept in sync with this
const int edgeSize = 3 * Pathfinding::NAVCELLS_PER_TILE; // number of tiles around the edge that will be off-world

This ticket can be solved in two separate patches:

1) The magic number should become a global constant.

2) The number must be taken into account in CCmpTerritoryManager.cpp as well, so that the territory line doesn't disappear in the shroud of darkness.

https://trac.wildfiregames.com/raw-attachment/ticket/4636/hidden_territory_at_map_border.jpg

(This might influence #4267)

Change History (2)

by elexis, 7 years ago

comment:1 by elexis, 7 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.