Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#849 closed defect (fixed)

FT: Animation glitch at the fuel tower

Reported by: eriktorbjorn Owned by: fingolfin
Priority: normal Component: Engine: SCUMM
Keywords: Cc:
Game: Full Throttle

Description

The animation of the hover-whatever at the fuel tower
is glitchy in the latest CVS, and has been for as long
as I can remember. Since I tried - unsuccessfully - to
fix it once, long ago, I already know more or less what
causes the problem:

The hover-whatever changes costume in mid-landing.
Since the actor is clearly visible, setActorCostume()
will hide the actor, reset the costume, and then show
the actor again. (I don't know why it does this, but
removing the hide/show lines caused glitches when I
tried it.)

The showActor() function will call adjustActorPos(),
which is what will cause the hover-whatever to move
almost out of the picture. (Removing the call to
adjustActorPos() will cause crashes, so that's not a
good idea either.)

I'm not sure how to fix this properly. Maybe
adjustActorPos() shouldn't be called when
costumeNeedsInit is true?

Ticket imported from: #749970. Ticket imported from: bugs/849.

Attachments (1)

ft.s52.gz (9.2 KB) - added by eriktorbjorn 16 years ago.
Savegame just before the animation starts

Download all attachments as: .zip

Change History (7)

comment:1 Changed 16 years ago by eriktorbjorn

Component: Engine: SCUMM
Game: Full Throttle

Changed 16 years ago by eriktorbjorn

Attachment: ft.s52.gz added

Savegame just before the animation starts

comment:2 Changed 16 years ago by fingolfin

What is the status of this item?

comment:3 Changed 16 years ago by eriktorbjorn

> What is the status of this item?

Unchanged. The glitch is still there.

comment:4 Changed 16 years ago by fingolfin

Status: newclosed

comment:5 Changed 16 years ago by fingolfin

Fixed this by extending the alternate setActorCostume(), written
for COMI/V8, to V7 games as well. I hope this won't cause
problems in DIG/FT. But in fact I am pretty confident it won't,
rather it might fix 1-2 other open bugs... but time will tell :-)

comment:6 Changed 16 years ago by fingolfin

Owner: set to fingolfin
Resolution: fixed
Note: See TracTickets for help on using tickets.