#7596 closed feature request (wontfix)
PALM: Memory leaks and misuse
| Reported by: | SF/eduardo-costa | Owned by: | SF/chrilith |
|---|---|---|---|
| Priority: | normal | Component: | Port: PalmOS |
| Version: | Keywords: | ||
| Cc: | Game: |
Description
I read that PalmOS backend does not work in Zire 71 and others because of limited heap. After some "dig" into the source code, I have a bug/feature request: 1. Use MemHandleNew instead of MemPtrNew 2. Use MHN or MPN instead of malloc
Since ScummVM for PalmOS uses a lot of "new" operations (which are based on "malloc"), Palm Memory Manager will never defragment memory.
Ticket imported from: #1825272. Ticket imported from: feature-requests/412.
Change History (5)
comment:1 by , 17 years ago
| Owner: | set to |
|---|
comment:2 by , 17 years ago
comment:3 by , 17 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
comment:4 by , 17 years ago
Hi,
I'm sorry to say that this is not possible :( In fact, MemHandleNew requires a memory lock/unlock system which is not implemented in any of the ScummVM engines. So, it will require lots of modification and will be a serious issue regarding performance.
new operator is not really based on malloc... anyway my malloc implementation is based on MemPtrNew/MemChunckNew and defragmentation occured only on PalmOS for MemHandles and only if they are unlocked.
comment:5 by , 6 years ago
| Component: | → Port: PalmOS |
|---|
This is not a bug, rather a "feature request"
Furthermore, while I am not that familiar with PalmOS coding, I believe this change would be far from trivial.