UFO: Alien Invasion Issue Tracker
star_faded.png
Please log in to bookmark issues
feature_request_small.png
OPEN  Feature request #4108  -  More fault-tolerant spawning
Posted Nov 17, 2012 - updated Feb 23, 2015
action_vote_minus_faded.png
0
Votes
action_vote_plus_faded.png
aduke1 (aduke1) has been working on this issue since January 28, 2013 (20:49)
Issue details
  • Type of issue
    Feature request
  • Status
     
    Accepted
  • Assigned to
     aduke1
  • Type of bug
    Not triaged
  • Likelihood
    Not triaged
  • Effect
    Not triaged
  • Posted by
     geever
  • Owned by
    Not owned by anyone
  • Category
    Engine
  • Priority
    1. Very low
  • Targetted for
    icon_milestones.png Not determined
  • Complexity
    icon_customdatatype.png Not determined
Issue description
Item 624 imported from sourceforge.net tracker on 2013-01-28 20:23:02

If the map has fewer spawn points than expected the server crashes. Instead it should print a noticeable error message and continue with less spawned characters.
Comments Ported from Sourceforge  ⇑ top
aduke1 (2012-11-17 22:23:31.167000)  ⇑ top
1. I didn't find the place where the crash/exit is caused. Do you have the exact msg ? 2. with "noticeable error message" you mean popup ? Like SysError(), just no crash ? 3. Had an idea: ufo has the capability to send crashdumps to our server. Should we add an option to the popup(s) to send error infos to us ??
aduke1 (2012-11-17 22:23:50.470000)  ⇑ top
- **assigned_to**: Duke
 
aduke1 (2012-11-17 22:56:18.640000)  ⇑ top
ad 1: The message is: "Not enough spawn points for team 1 (actorsize: 1)" issued from G_ClientTeamInfo()
aduke1 (2012-11-17 23:56:05.763000)  ⇑ top
It would be easy to allow some threshold in G_ClientTeamInfo(), like "if just ONE spawnpoint is missing, continue anyway", BUT - not sure what the client will do (see segfault in Bug report 3424 - not enough spawnpoints leads to segfault) - players will NOT accept less soldiers in battle than there were on the aircraft.

==> imho our best bet is to find better ways to make sure that there are enough spawnpoints
shipit (2012-11-20 10:11:24.161000)  ⇑ top
There should be no problem with the spawnpoints for the human team in campaign. This one occoured because the base map was not properly updated after we added the 8+ soldiers support.

Iirc the map tests already check for multiplayer spawnpoints if mp is enabled in the mapdef. Maybe we can add a check for human spawnpoints, too?
natewr (2013-01-22 15:14:57.255000)  ⇑ top
- **status**: open --> accepted