Opened 19 months ago

Last modified 17 months ago

#10221 new defect

SCI: KQ7 v1.4 Bad view priority, Ego behind curtain, Chapter 5

Reported by: dafioram Owned by:
Priority: low Component: Engine: SCI
Keywords: sci32 Cc:
Game: King's Quest 7

Description

Game: KQ7 v1.4
OS: win7-64
scummvm: 1.10.0git-4882-g1504f41

Doesn't happen in v2.00b. At the beginning of chapter 5 after giving the statue to the snake you can walk behind his curtain.

Attachments (4)

scummvm00000.png (261.3 KB) - added by dafioram 19 months ago.
KQ7_1.4.png (159.3 KB) - added by dafioram 19 months ago.
save.zip (42.1 KB) - added by dafioram 19 months ago.
dafioram-diff.txt (232.3 KB) - added by m-kiewitz 19 months ago.
Dafioram's diff

Download all attachments as: .zip

Change History (12)

Changed 19 months ago by dafioram

Attachment: scummvm00000.png added

comment:1 Changed 19 months ago by m-kiewitz

Does this also happen, when using the original interpreter?

Changed 19 months ago by dafioram

Attachment: KQ7_1.4.png added

Changed 19 months ago by dafioram

Attachment: save.zip added

comment:2 Changed 19 months ago by dafioram

Yes.

comment:3 Changed 19 months ago by csnover

Summary: SCI: KQ7 Bad palette priority, Ego behind curtain, Chapter 5SCI: KQ7 Bad view priority, Ego behind curtain, Chapter 5

comment:4 Changed 19 months ago by csnover

Keywords: sci32 added

comment:5 Changed 19 months ago by dafioram

If I compare 1.4 vs 2.0b I see that in script/room 5300 the priority for the curtain object is 98 (1.4) vs 90 (2.0b) this probably doesn't matter though since the KQEgo priority (owned by script 13) has a priority of 109 at this point.

The only other difference I see for the properties of the curtain is that -methDict- is 466 (1.4) vs. 294 (2.0b).

curtain setPri are the same.

comment:6 Changed 19 months ago by dafioram

Here is the two files the first revision is the 5300 file for 1.4 and the 2nd is for 2.0b. I did a full decompile of all files so function names and such should be resolved.

https://gist.github.com/dafioram/5ce7c5d14d88dd98128b04da07c79073

I was hoping that the gist would give me a diff side by side, but i guess the files are too long.

comment:7 Changed 19 months ago by dafioram

Here is a better diff, shows 1.4->2.0b for script 5300.
https://pastebin.com/SWCKxgKa

Changed 19 months ago by m-kiewitz

Attachment: dafioram-diff.txt added

Dafioram's diff

comment:8 Changed 17 months ago by dafioram

Summary: SCI: KQ7 Bad view priority, Ego behind curtain, Chapter 5SCI: KQ7 v1.4 Bad view priority, Ego behind curtain, Chapter 5
Note: See TracTickets for help on using tickets.