Opened 13 years ago

Closed 13 years ago

Last modified 9 months ago

#8537 closed patch

residual: improve text object resource handling

Reported by: SF/mannythegnome Owned by: aquadran
Priority: normal Component: ResidualVM
Keywords: Cc:
Game:

Description

The attached patch ensures that the resources
associated with a killed text object are destroyed
(i.e. the bitmap) which prevents residual from becoming
a resource hog in the main menu.

I'm not sure whether the delete statement is
appropriate at that place, and I don't know enough CPP
to judge whether the remove statement is superfluous,
i.e. whether the delete statement implies removal from
lists.

The patch also removes the TextObjectExists statement
from BlastText(), which IMHO isn't neccessary (the
object will be destroyed right after it was drawn).

Ticket imported from: #1495487. Ticket imported from: patches/642.

Attachments (1)

residual-text.diff (907 bytes) - added by SF/mannythegnome 13 years ago.
proposed patch

Download all attachments as: .zip

Change History (10)

Changed 13 years ago by SF/mannythegnome

Attachment: residual-text.diff added

proposed patch

comment:1 Changed 13 years ago by SF/mannythegnome

Hrm the actor.cpp code seem to call delete itself for each
sayline deletion, while the deleteTextObjects code of the
engine (that deletes all the text objects) does the deletion.

comment:2 Changed 13 years ago by fingolfin

Owner: set to aquadran

comment:3 Changed 13 years ago by fingolfin

aquadran, do you think this patch can / should be applied?

comment:4 Changed 13 years ago by aquadran

i didn't have a time for look close into that patch yet. So
i don't know yet.

comment:5 Changed 13 years ago by fingolfin

Any news on this?

comment:6 Changed 13 years ago by sev-

What is the status of this item?

comment:7 Changed 13 years ago by aquadran

stalled, i need look close into changes

comment:8 Changed 13 years ago by aquadran

Status: newclosed

comment:9 Changed 9 months ago by digitall

Component: ResidualVM
Note: See TracTickets for help on using tickets.