Opened 17 years ago

Closed 17 years ago

Last modified 13 years ago

#387 closed defect (fixed)

SIMON: not wearing beard

Reported by: wjp Owned by: fingolfin
Priority: normal Component: Engine: AGOS
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, 17 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, 17 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, 17 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, 17 years ago

Anyone had any luck with this one?

comment:5 by eriktorbjorn, 17 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, 17 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, 17 years ago

Owner: set to fingolfin
Resolution: fixed
Status: newclosed

comment:8 by fingolfin, 17 years ago

Willem's patch is in CVS now.

comment:9 by Kirben, 17 years ago

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

comment:10 by Kirben, 17 years ago

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

comment:11 by fingolfin, 15 years ago

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

comment:12 by Kirben, 13 years ago

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