cpp/openttd-patchpack/source Changeset - r28334:e41ee4d6fc7c

Changeset - r28334:e41ee4d6fc7c

Parent rev.

Child rev.

[Not reviewed]

master

0 5 0

Peter Nelson - 4 months ago 2023-12-25 20:59:37
peter1138@openttd.org

Fix 9ce1626b: Some blitters have `bp->remap` aliased to `remap` for performance. (#11626)

While this probably doesn't make a huge difference for the custom transparent remap code path, the alias is there so use it.

5 files changed with 6 insertions and 6 deletions:

src/blitter/32bpp_anim.cpp

src/blitter/32bpp_anim_sse4.cpp

src/blitter/32bpp_optimized.cpp

src/blitter/32bpp_sse_func.hpp

src/blitter/40bpp_anim.cpp

0 comments (0 inline, 0 general)

src/blitter/32bpp_anim.cpp

➞

Show inline comments

@@ @@ -218,7 +218,7 @@ inline void Blitter_32bppAnim::Draw(cons @@
 					if (src_px->a != 0) {
 						src_px += n;
 						do {
-							*dst = this->LookupColourInPalette(bp->remap[GetNearestColourIndex(*dst)]);
 							*dst = this->LookupColourInPalette(remap[GetNearestColourIndex(*dst)]);
 							*anim = 0;
 							anim++;
 							dst++;

src/blitter/32bpp_anim_sse4.cpp

➞

Show inline comments

@@ @@ -322,7 +322,7 @@ bmcr_alpha_blend_single: @@
 				/* Apply custom transparency remap. */
 				for (uint x = (uint) bp->width; x > 0; x--) {
 					if (src->a != 0) {
-						*dst = this->LookupColourInPalette(bp->remap[GetNearestColourIndex(*dst)]);
 						*dst = this->LookupColourInPalette(remap[GetNearestColourIndex(*dst)]);
 						*anim = 0;
+					}
 					src_mv++;

src/blitter/32bpp_optimized.cpp

➞

Show inline comments

@@ @@ -209,7 +209,7 @@ inline void Blitter_32bppOptimized::Draw @@
 					if (src_px->a != 0) {
 						src_px += n;
 						do {
-							*dst = this->LookupColourInPalette(bp->remap[GetNearestColourIndex(*dst)]);
 							*dst = this->LookupColourInPalette(remap[GetNearestColourIndex(*dst)]);
 							dst++;
 						} while (--n != 0);
 					} else {

src/blitter/32bpp_sse_func.hpp

➞

Show inline comments

@@ @@ -396,7 +396,7 @@ bmcr_alpha_blend_single: @@
 				/* Apply custom transparency remap. */
 				for (uint x = (uint) bp->width; x > 0; x--) {
 					if (src->a != 0) {
-						*dst = this->LookupColourInPalette(bp->remap[GetNearestColourIndex(*dst)]);
 						*dst = this->LookupColourInPalette(remap[GetNearestColourIndex(*dst)]);
+					}
 					src_mv++;
 					dst++;

src/blitter/40bpp_anim.cpp

➞

Show inline comments

@@ @@ -267,9 +267,9 @@ inline void Blitter_40bppAnim::Draw(cons @@
 						src_px += n;
 						do {
 							if (*anim != 0) {
-								*anim = bp->remap[*anim];
 								*anim = remap[*anim];
 							} else {
-								*dst = this->LookupColourInPalette(bp->remap[GetNearestColourIndex(*dst)]);
 								*dst = this->LookupColourInPalette(remap[GetNearestColourIndex(*dst)]);
 								*anim = 0;
+							}
 							anim++;

0 comments (0 inline, 0 general)