Opened 17 years ago

Closed 17 years ago

Last modified 17 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 17 years ago.
Savegame just before the animation starts

Download all attachments as: .zip

Change History (7)

comment:1 by eriktorbjorn, 17 years ago

Component: Engine: SCUMM
Game: Full Throttle

by eriktorbjorn, 17 years ago

Attachment: ft.s52.gz added

Savegame just before the animation starts

comment:2 by fingolfin, 17 years ago

What is the status of this item?

comment:3 by eriktorbjorn, 17 years ago

> What is the status of this item?

Unchanged. The glitch is still there.

comment:4 by fingolfin, 17 years ago

Status: newclosed

comment:5 by fingolfin, 17 years ago

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 by fingolfin, 17 years ago

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