Comment 37 for bug 895529

Revision history for this message
Bug Importer (bug-importer) wrote :

Code related to this issue has just been checked in!
Author: DCoder
Location: trunk, r1141
Commit contains DLL: Yes
Revision comment:
Related to issue #917 - ugh, this again... Quite stupidly, the check for BaseUnit was placed in the base plan handler, which happens *after* the house acquires and deploys an MCV, which requires it to have something buildable from BaseUnit already. Awesome.
The first house failing to find BaseUnit will print a notice about it, the following ones won't - this is due to the game's internal organization, rearranging it is not a good idea right now. The game shouldn't crash, but if it does, the logging for the first house should be sufficient. If it does crash, the usual generic error message will be shown, not a special one with explanations about the possible causes - we decided that Ares shouldn't force a crash if this happens, so we cannot tell if a crash two seconds later is related to this problem or not.

Additional improvement: if parser errors have been detected, the generic IE window will mention this fact. Though WW's own code causes them by default, hoorah.
SVN: http://svn.renegadeprojects.com/Ares/1141