SCI: Updated some comments related to Freddy Pharkas and silenced warning for kDoAudio subop 13

svn-id: r51499
This commit is contained in:
Filippos Karapetis 2010-07-30 07:11:57 +00:00
parent 7c190074c1
commit a9a0451b15
2 changed files with 6 additions and 5 deletions

View File

@ -177,7 +177,7 @@ static reg_t kSetCursorSci11(EngineState *s, int argc, reg_t *argv) {
case 10: // Freddy Pharkas, when using the whiskey glass to read
// the prescription (bug #3034973) - looks like an erroneous
// extra param
// TODO: We don't really use parameter indices 6-10 (with no side
// TODO: We don't really use parameter indices 5-9 (with no side
// effects), are they used for something, or are these a result of
// script bugs?
hotspot = new Common::Point(argv[3].toSint16(), argv[4].toSint16());

View File

@ -222,16 +222,17 @@ reg_t kDoAudio(EngineState *s, int argc, reg_t *argv) {
// 3 new subops in Pharkas. kDoAudio in Pharkas sits at seg026:038C
case 11:
// Not sure where this is used yet
warning("kDoAudio: Unhandled case 11, %d extra arguments passed", argc - 1);
break;
case 12:
// Seems to be audio sync, used in Pharkas. Silenced the warning due to
// the high level of spam it produces.
// Seems to be some sort of audio sync, used in Pharkas. Silenced the
// warning due to the high level of spam it produces. (takes no params)
//warning("kDoAudio: Unhandled case 12, %d extra arguments passed", argc - 1);
break;
case 13:
// Used in Pharkas whenever a speech sample starts
warning("kDoAudio: Unhandled case 13, %d extra arguments passed", argc - 1);
// Used in Pharkas whenever a speech sample starts (takes no params)
//warning("kDoAudio: Unhandled case 13, %d extra arguments passed", argc - 1);
break;
default:
warning("kDoAudio: Unhandled case %d, %d extra arguments passed", argv[0].toUint16(), argc - 1);