Is this a mistake in CVS source?

#1 Mar 14, 2011, 3:58 am Hide

fejoa

In char.cpp at the bottom:

void character::Slow()
{
  doforbodyparts()(this, &bodypart::Slow);
  doforequipments()(this, &item::Slow);
  BeginTemporaryState(HASTE, 500 + RAND() % 1000);
}

When Slow() is called, it will slow equipments and bodyparts, but activate Haste as a temporary state. Should it instead be:

void character::Slow()
{
  doforbodyparts()(this, &bodypart::Slow);
  doforequipments()(this, &item::Slow);
  BeginTemporaryState(SLOW, 500 + RAND() % 1000);
}

?

Either an elder dark mage, or

should be able to verify that this is a bug, when they cast "slow" on the player.

#2 Mar 14, 2011, 5:41 am Hide

Knowing the twisty twists and uncommented code of IVAN, this might as well be a deliberate feature (maybe so that slows don't stack their effect, or something?). But yeah, gotta change 'haste' to 'slow' and do the testing.

#3 Mar 14, 2011, 7:34 am Hide