Difficulty bug, perhaps.

[kevL]

introducing 2 muton soldiers

The first has been mind-controlled this turn, and its stats show normal, adjusted for Genius difficulty.

The second, however, was mind-controlled last turn and reloaded from a save. Its stats have reverted to Beginner.


I tried to hunt this down but it eludes me; the closest i got is that UnitStats seem to be derived via BattleUnit for fresh aliens, while reloaded aliens appear to take theirs from Unit

( it's slightly possible there's something quirky in my ruleset, so confirmation is appreciated )



These screens were taken in git_master_2013_10_17_1853 (recent)

[kevL]

ps.
I also noticed that BattleUnit::adjustStats() lists strength twice, resulting in a double bonus for Difficulty. (don't want to commit on that until this is sorted tho)

[Shadow]

Might want to put this on the bug tracker.

[kevL]

I'll give it a day and if no one bites will open a reference.

i feel there can be a more open discussion, which i'm interested in learning from, here on the forum. Things are pretty claustrophobic -- open and shut -- on the 'tracker

[kevL]

ok found it.

The BattleUnit class has two constructors, one for soldiers, the second for aliens. Only the aliens make a call to adjustStats(), which levels them up per Difficulty. unfortunately this bit of protective code:

Code: [Select]

if (faction == FACTION_HOSTILE)
{
adjustStats(diff);
}
in the second constructor prevents mind controlled aliens *that are reloaded* while Mc'd from getting their difficulty bonuses. Because units are saved to file with their FACTION value: 0 for soldiers, 1 for aliens. Hence even though Mc'd aliens still use the second cTor, that bit there thinks they are Faction_Player...

ie. by commenting out the condition, things returned to what I think they should be. But who knows, maybe others want mind-controlled aliens that are reloaded to be Beginners ....


workaround: do not save reload while having an Mc'd alien.


tl;dr It takes someone with more knowledge of what those 2 constructors do than me, to make change.

[ edit ] the issue is in the bug traker..

[Warboy1982]

simply commenting out that code would result in xcom tanks and civilians being adjusted as well.
i've added a call to adjustStats() in the load function that should fix the problem.

[kevL]

ok. Thanks a lot, Wb.

will let you know if (i notice) things go bork