aboutsummaryrefslogtreecommitdiff
path: root/engines/sci/engine/kvideo.cpp
diff options
context:
space:
mode:
authorLars Skovlund2011-02-13 22:34:29 +0100
committerLars Skovlund2011-02-13 22:34:29 +0100
commita5b198b23337e3b4f44dabefccb9f097b37b3198 (patch)
tree4d40b9eb62c8a36c9e294f0d2602343a156ae71e /engines/sci/engine/kvideo.cpp
parent2087e2325bd97a31c3518d87918ba87727faff3c (diff)
downloadscummvm-rg350-a5b198b23337e3b4f44dabefccb9f097b37b3198.tar.gz
scummvm-rg350-a5b198b23337e3b4f44dabefccb9f097b37b3198.tar.bz2
scummvm-rg350-a5b198b23337e3b4f44dabefccb9f097b37b3198.zip
Remove erroneous comment. It reflected a false belief resulting from
a missing parameter check, which has also been added.
Diffstat (limited to 'engines/sci/engine/kvideo.cpp')
-rw-r--r--engines/sci/engine/kvideo.cpp6
1 files changed, 2 insertions, 4 deletions
diff --git a/engines/sci/engine/kvideo.cpp b/engines/sci/engine/kvideo.cpp
index 155dec5e27..0706da0d9e 100644
--- a/engines/sci/engine/kvideo.cpp
+++ b/engines/sci/engine/kvideo.cpp
@@ -236,10 +236,8 @@ reg_t kPlayVMD(EngineState *s, int argc, reg_t *argv) {
case 0: // init
s->_videoState.reset();
s->_videoState.fileName = s->_segMan->derefString(argv[1]);
- // TODO: argv[2] (usually null). When it exists, it points to an "Event" object,
- // that holds no data initially (e.g. in the intro of Phantasmagoria 1 demo).
- // Perhaps it's meant for syncing
- if (argv[2] != NULL_REG)
+
+ if (argc > 2 && argv[2] != NULL_REG)
warning("kPlayVMD: third parameter isn't 0 (it's %04x:%04x - %s)", PRINT_REG(argv[2]), s->_segMan->getObjectName(argv[2]));
break;
case 1: