Project

General

Profile

Actions

Feature #1505

closed

MaxTopUnitsOnTile

Added by Marko Lindqvist about 2 months ago. Updated 19 days ago.

Status:
Closed
Priority:
Normal
Category:
General
Target version:
Start date:
06/10/2025
Due date:
% Done:

0%

Estimated time:

Description

Add requirement type "MaxTopUnitsOnTile". The difference to old "MaxUnitsOnTile" is that transported units are not counted, only topmost transporters.


Files


Related issues 1 (0 open1 closed)

Blocks Feature #1517: granularity: Allow max 4 units on tileClosedMarko Lindqvist06/12/2025

Actions
Actions #1

Updated by Marko Lindqvist about 2 months ago

Actions #2

Updated by Alexandr Ignatiev about 2 months ago

Briefly looking, seems like in reqtext.c MAXTILETOPUNITS has inherited the doc from MAXTILEUNITS?

Actions #3

Updated by Marko Lindqvist about 2 months ago

Alexandr Ignatiev wrote in #note-2:

Briefly looking, seems like in reqtext.c MAXTILETOPUNITS has inherited the doc from MAXTILEUNITS?

Yes. It's more correct for MAXTILETOPUNITS than MAXTILETOTALUNITS (renamed MAXTILEUNITS), for which the doc has been changed.

Actions #4

Updated by Alexandr Ignatiev about 2 months ago

to

Marko Lindqvist wrote in #note-3:

Alexandr Ignatiev wrote in #note-2:

Briefly looking, seems like in reqtext.c MAXTILETOPUNITS has inherited the doc from MAXTILEUNITS?

Yes. It's more correct for MAXTILETOPUNITS than MAXTILETOTALUNITS (renamed MAXTILEUNITS), for which the doc has been changed.

Hmm... If I do not miss some patch, it said about number of units present on the tile(s). Really, the difference between "number of units" and "total number of units" does not sound very clear. At least we are used to that a unit that is inside Y which is in X is still present in X.

Actions #5

Updated by Marko Lindqvist about 2 months ago

  • Status changed from In Review to In Progress
Actions #6

Updated by Marko Lindqvist about 2 months ago

I changed MaxTopUnitOnTile reqtext.c messages to talk about units directly on the tile. Is this better?

Actions #7

Updated by Alexandr Ignatiev about 2 months ago

Marko Lindqvist wrote in #note-6:

I changed MaxTopUnitOnTile reqtext.c messages to talk about units directly on the tile. Is this better?

Well, I'd use something like "untransported" or "(except ones in transports)", but tastes differ, it's already something :)

Actions #8

Updated by Marko Lindqvist about 2 months ago

  • Blocks Feature #1517: granularity: Allow max 4 units on tile added
Actions #9

Updated by Marko Lindqvist about 1 month ago

  • Status changed from In Review to In Progress

You're right "untransported" would be better. "directly on tile" is somewhat ambiguous when it comes to units inside cities or extras (fortresses).

Actions #10

Updated by Marko Lindqvist about 1 month ago

Marko Lindqvist wrote in #note-9:

You're right "untransported" would be better. "directly on tile" is somewhat ambiguous when it comes to units inside cities or extras (fortresses).

Changed the patch accordingly.

Actions #11

Updated by Marko Lindqvist 19 days ago

  • Status changed from In Review to Closed
Actions

Also available in: Atom PDF