UFO: Alien Invasion Issue Tracker
UFO: Alien Invasion
Go to the previous open issue
Go to the previous issue (open or closed)
star_faded.png
Please log in to bookmark issues
icon_project.png UFO: Alien Invasion / Closed Bug report #2259 R_ModBeginLoading: invalid positions
Go to the next issue (open or closed)
Go to the next open issue
This issue has been closed with status "Closed" and resolution "Not determined".
Issue basics
  • Type of issue
    Bug report
  • Category
    Battlescape
  • Targetted for
    Not determined
  • Status
    Closed
  • Priority
    1. Very low
User pain
  • Type of bug
    Not triaged
  • Likelihood
    Not triaged
  • Effect
    Not triaged
Affected by this issue (1)
People involved
Times and dates
  • Posted at
  • Last updated
  • Estimated time
    Not estimated
Issue details
  • Resolution
    Not determined
  • Reproducability
    Not determined
  • Severity
    Not determined
  • Complexity
    Not determined
  • Platform
    Not determined
  • Architecture
    Not determined
Attachments (0)
There is nothing attached to this issue
Duplicate issues (0)
This issue does not have any duplicates
Description
[http://sourceforge.net/p/ufoai/bugs/2259 Item 2259] imported from sourceforge.net tracker on 2013-01-28 19:26:38

I created a +forest huge assembly with a size of 16x16 for RMA performance testing.
The assembly worked, the CM_LoadMap string looks good, but later we get the above problem when trying to actually load the map, probably because the position string (or both ?) has been truncated somewhere.

A 16x16 RMA is not really playable, so this is a 'not in 2.3' issue. But if somebody happens to know how to overcome this limitation, it would help me with rma preformance. Keep in mind that RMA supports up to 32x32 maps.

The minimum solution would be a better error msg like "string too long for...".
===== Comments Ported from Sourceforge =====

====== aduke1 (2010-01-03 10:20:08) ======

log
====== tlh2000 (2010-01-03 10:42:13) ======

this has to do with the config string limitation - keep in mind that the rma stuff also works for multiplayer - that's why all the strings of the tiles and positions must be send to the clients, this is achieved by the configstring mechanics. and they have a max length.
====== aduke1 (2010-01-03 11:27:46) ======

Yes, that's how far I got yesterday ;)
But what is that max length ?? Is it MAX_TOKEN_CHARS 256 ? (probably)
Or MAX_TILESTRINGS * MAX_TOKEN_CHARS ? (I don't think so).


====== tlh2000 (2010-01-03 18:23:41) ======

for the tiles and positions it's MAX_TILESTRINGS * MAX_TOKEN_CHARS for all other configstrings it's MAX_TOKEN_CHARS
====== aduke1 (2010-01-03 19:44:55) ======

Hmm. That would be 25x256, >6000
In the attached log, the TOKEN string is less than 500 and the POSITON string less than 900 long.
On a regular RMA (eg. industrial), both strings are less than 256 bytes long.
Maybe the splitting algo doesn't work correctly ?
====== tlh2000 (2010-01-04 07:12:07) ======

afair this isn't splitted - see SV_SpawnServer:

Q_strncpyz(sv.configstrings[CS_TILES], server, MAX_TOKEN_CHARS * MAX_TILESTRINGS);
Q_strncpyz(sv.configstrings[CS_POSITIONS], param, MAX_TOKEN_CHARS * MAX_TILESTRINGS);
====== aduke1 (2011-11-25 22:39:05) ======

The priority was adjusted according to the "bug priority rules". See
http://ufoai.ninex.info/wiki/index.php/Bugs#Bug_Priorities
====== aduke1 (2011-11-25 22:39:06) ======

Fixed Oct 26 2011.
Steps to reproduce this issue
Nothing entered.
Todos (0 / 0)
Issue created
footer_logo.png The Bug Genie 4.3.1 | Support | Feedback spinning_16.gif