Opened 21 years ago
Closed 21 years ago
Last modified 4 years ago
#8055 closed patch
Further drawBomp() cleanups
|Reported by:||eriktorbjorn||Owned by:||aquadran|
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.
Change History (5)
by , 21 years ago
comment:1 by , 21 years ago
|Status:||new → closed|
comment:2 by , 21 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 , 21 years ago
comment:4 by , 4 years ago
|Component:||→ Engine: SCUMM|
Patch against a September 17 CVS snapshot