[Scummvm-cvs-logs] SF.net SVN: scummvm:[51484] scummvm/trunk/engines/sci/engine/kmovement.cpp
m_kiewitz at users.sourceforge.net
m_kiewitz at users.sourceforge.net
Fri Jul 30 00:04:23 CEST 2010
Revision: 51484
http://scummvm.svn.sourceforge.net/scummvm/?rev=51484&view=rev
Author: m_kiewitz
Date: 2010-07-29 22:04:23 +0000 (Thu, 29 Jul 2010)
Log Message:
-----------
SCI: reverting r51479 and adding TODOs for fixing the hoyle issue (fixing this properly should make credits etc. appear and solve all the other issues that clone submitted)
Modified Paths:
--------------
scummvm/trunk/engines/sci/engine/kmovement.cpp
Modified: scummvm/trunk/engines/sci/engine/kmovement.cpp
===================================================================
--- scummvm/trunk/engines/sci/engine/kmovement.cpp 2010-07-29 21:33:42 UTC (rev 51483)
+++ scummvm/trunk/engines/sci/engine/kmovement.cpp 2010-07-29 22:04:23 UTC (rev 51484)
@@ -328,6 +328,8 @@
bool collision = false;
reg_t cantBeHere = NULL_REG;
+ // FIXME here -> cantBeHere detection doesn't work in hoyle 3
+ // it's using kCanBeHere but it just has a selector called cantBeHere, so this here doesn't work
if (SELECTOR(cantBeHere) != -1) {
invokeSelector(s, client, SELECTOR(cantBeHere), argc, argv);
if (!s->r_acc.isNull())
@@ -349,17 +351,14 @@
debugC(2, kDebugLevelBresen, "Finished mover %04x:%04x by collision", PRINT_REG(mover));
// We shall not set completed in this case, sierra sci also doesn't do it
// if we set call .moveDone in those cases qfg1 vga gate at the castle and lsl1 casino door will not work
- // Update: however, it seems that Hoyle 3 (full and demo) does need to end here.
- // TODO/FIXME: Find out why Hoyle 3 needs this.
- // The following fixes bugs #3035077, #3035080, #3035081 and #3035242
- if (g_sci->getGameId() == GID_HOYLE3)
- completed = 1;
}
if ((getSciVersion() >= SCI_VERSION_1_EGA))
if (completed)
invokeSelector(s, mover, SELECTOR(moveDone), argc, argv);
+ // FIXME here -> cantBeHere detection doesn't work in hoyle 3
+ // it's using kCanBeHere but it just has a selector called cantBeHere, so this here doesn't work
if (SELECTOR(cantBeHere) != -1)
return cantBeHere;
return make_reg(0, completed);
This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.
More information about the Scummvm-git-logs
mailing list