#8055 closed patch
Further drawBomp() cleanups
| Reported by: | eriktorbjorn | Owned by: | aquadran |
|---|---|---|---|
| Priority: | normal | Component: | Engine: SCUMM |
| Version: | Keywords: | ||
| Cc: | Game: |
Description
This patch further cleans up the new drawBomp() implementation. It removes one theoretical - but extremely unlikely - memory leak, slightly optimizes the image scaling, and removes the floating-point math from the blending function.
There are still some FIXMEs, and some places in the code - e.g. drawBlastObject() - which assume that drawBomp() can't scale images, but this shouldn't cause any visible harm.
Ticket imported from: #610954. Ticket imported from: patches/160.
Attachments (1)
Change History (5)
by , 22 years ago
| Attachment: | drawbomp2.diff added |
|---|
comment:1 by , 22 years ago
| Status: | new → closed |
|---|
comment:2 by , 22 years ago
If you accept a patch, please make sure to assign the tracker item to you, to clearly flag you as the person who reviewed & accepted the patch, and thus also is responsible for any regressions caused by it.
OK in this case aquadran was not setup as a tracker technican, so he couldn't have assigned this to himself :-)
comment:3 by , 22 years ago
| Owner: | set to |
|---|
comment:4 by , 5 years ago
| Component: | → Engine: SCUMM |
|---|
Patch against a September 17 CVS snapshot