Skip to content

Commit

Permalink
SCI: Clarified comment in rev 5654e12
Browse files Browse the repository at this point in the history
It's a bad idea to write comments when half asleep :P
  • Loading branch information
bluegr committed Apr 28, 2011
1 parent b95ea8d commit d855092
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions engines/sci/sound/soundcmd.cpp
Expand Up @@ -345,8 +345,9 @@ reg_t SoundCommandParser::kDoSoundFade(int argc, reg_t *argv, reg_t acc) {
musicSlot->fadeStep = volume > musicSlot->fadeTo ? -5 : 5;
musicSlot->fadeTickerStep = argv[2].toUint16() * 16667 / _music->soundGetTempo();
musicSlot->fadeTicker = 0;
// TODO: We only handle zero and non-zero parameters, but this parameter
// can have other values as well (e.g. it's 3 in KQ6).
// TODO: We handle this as a bit field (i.e. containing values 0 and 1),
// but some games pass other values here as well (e.g. some KQ6 scripts
// pass 3 here)
musicSlot->stopAfterFading = (argc == 5) ? (argv[4].toUint16() != 0) : false;

// WORKAROUND/HACK: In the labyrinth in KQ6, when falling in the pit and
Expand Down

0 comments on commit d855092

Please sign in to comment.