Opened 22 years ago

Closed 22 years ago

Last modified 18 years ago

#387 closed defect (fixed)

SIMON: not wearing beard

Reported by: wjp Owned by: fingolfin
Priority: normal Component: Engine: AGOS
Version: Keywords:
Cc: Game: Simon the Sorcerer 1

Description

(Simon the Sorcerer 1, Linux)

When you 'wear' the dwarf's beard, it doesn't show on your character permanently.

I'm pretty sure the flag for 'wearing beard' is set by SET_CHILD2_BIT <151> 17

(I'll look into it further to see if I can figure anything else out)

Ticket imported from: #590800. Ticket imported from: bugs/387.

Change History (12)

comment:1 by SF/ender, 22 years ago

This might also have something to do with the fact the script tries to load 1641.VGA (which doesn't exist) just before wearing the beard. As we can't find the script, I added a patch to just warn and continue. It's possible that bit of code is actually important tho.

(See bug 558589, and my simon.cpp change in rev 1.29 vs 1.28)

comment:2 by eriktorbjorn, 22 years ago

I've seen this happen in the demo, where indeed it was because ScummVM was trying to load the wrong VGA file. The full game worked though but there seem to be several different versions of it, so I probably don't have the same as you do. (There were two different versions on the CD I bought, so who knows how many others there might be?)

comment:3 by SF/ender, 22 years ago

I don't actually have simon, so I just took other peoples word that the file was missing :P

The fact that none of the active scummvm team owns Simon (nor really care about it) is the reason there are so many open Simon bugs :)

comment:4 by SF/ender, 22 years ago

Anyone had any luck with this one?

comment:5 by eriktorbjorn, 22 years ago

All I know is that in the demo, the Simon-with-a-beard graphics seem to be in 0119.VGA instead of 1641.VGA. The complete versions of Simon that I bought don't contain any .VGA files at all.

comment:6 by wjp, 22 years ago

Funny, loading 0119.vga instead of 1641.vga works in my simon1dos too :-)

I'll create a small patch and submit it

comment:7 by fingolfin, 22 years ago

Owner: set to fingolfin
Resolution: fixed
Status: newclosed

comment:8 by fingolfin, 22 years ago

Willem's patch is in CVS now.

comment:9 by Kirben, 22 years ago

Component: Engine: AGOS
Game: Simon the Sorcerer 1
Owner: fingolfin removed
Status: closednew

comment:10 by Kirben, 22 years ago

Component: Engine: AGOS
Game: Simon the Sorcerer 1
Owner: set to fingolfin
Status: newclosed

comment:11 by fingolfin, 19 years ago

Component: Engine: AGOSEngine: AGI
Game: Simon the Sorcerer 1Space Quest 1

comment:12 by Kirben, 18 years ago

Component: Engine: AGIEngine: AGOS
Game: Space Quest 1Simon the Sorcerer 1
Note: See TracTickets for help on using tickets.