[comp.sources.games.bugs] Xconq 5 for X10 - Complaints and bugs

cnc@hpcilzb.HP.COM (Chris Christensen) (07/07/88)

>  And in "beirut.per" the density of buildings is so
> high it is impossible to move the car bombs and tanks around. And
> that one is particularly susceptible to segementation violations.
> If anyone has practical versions of these, they would be greatly
> appreciated. Otherwise, we may come up with our own.

Don't run the beirut period, run the senerio.
There is a bug that when the buildings defect it will core dump.
I have sent a fix to Stan. 

Basically the order of things in unit_revolt was not correct:

unit_revolt(unit)
Unit *unit;
{
    int u = unit->type, maxmor, chance;
    Side *oldside = unit->side;

    if (utypes[u].revolt > 0) {
        chance = utypes[u].revolt;
        maxmor = utypes[u].maxmorale;
        if (maxmor > 0) chance = (chance * (maxmor - unit->morale)) / maxmor;
        if (unit->trueside != oldside && random(10000) < chance) {
            notify(oldside, "%s revolts!", unit_handle(oldside, unit));
            see_exact(oldside, unit->x, unit->y);
            draw_hex(oldside, unit->x, unit->y, TRUE);
            all_see_hex(unit->x, unit->y);
            unit_changes_side(unit, unit->trueside, CAPTURE, DISASTER);
        }
    }
}

Actually if you add:

true B neutral

to the period file beirut.per this should also fix the bug by allowing
buildings to become neutral again.

> 4) It would be nice if we received some warning before trying
> to move a unit with no supplies out of a town - currently the
> unit dies immediately, often without the player realizing what
> happened.
> 
I will second this!