Opened 11 years ago

Closed 11 years ago

Last modified 15 months ago

#8919 closed patch

Saga engine reduced memory usage

Reported by: agent-q Owned by: bluegr
Priority: normal Component: Engine: SAGA
Keywords: Cc:
Game:

Description

Hi,

Here is a patch that (may) free enough memory to get Inherit the Earth to run on the DS in 0.12.0. The existing code stores the sample data in memory twice for many sample formats, and this code prevents that.

Please check it thoroughly! It works for me, but I have only tested one sample format.

This patch is based on the current 0.12.0 revision.

- Neil

Ticket imported from: #2100962. Ticket imported from: patches/1024.

Attachments (1)

itepatch (4.5 KB ) - added by agent-q 11 years ago.
Inherit the Earth: Memory reduction in SndRes::load()

Download all attachments as: .zip

Change History (5)

by agent-q, 11 years ago

Attachment: itepatch added

Inherit the Earth: Memory reduction in SndRes::load()

comment:1 by agent-q, 11 years ago

Owner: set to bluegr

comment:2 by bluegr, 11 years ago

Thanks for your patch Neil, indeed your code handles sound samples in a better way, without loading them twice (oops...)

I've tested this with all the versions of ITE I got, plus IHNM, and no problems were caused

Committed as-is

comment:3 by bluegr, 11 years ago

Status: newclosed

comment:4 by digitall, 15 months ago

Component: Engine: SAGA
Note: See TracTickets for help on using tickets.