cpp/openttd-patchpack/source Changeset - r24989:aa99b7b829d9

Changeset - r24989:aa99b7b829d9

Parent rev.

Child rev.

[Not reviewed]

master

0 3 0

Michael Lutz - 4 years ago 2021-03-02 19:55:39
michi@icosahedron.de

Fix #8774: Black screenshots when using 40bpp-blitter. (#8791)

This affected all screenshot types that render to an off-screen
buffer and don't copy the actual screen contents.

3 files changed with 45 insertions and 22 deletions:

src/blitter/32bpp_optimized.cpp

src/blitter/32bpp_optimized.hpp

src/blitter/40bpp_anim.cpp

0 comments (0 inline, 0 general)

src/blitter/32bpp_optimized.cpp

➞

Show inline comments

@@ @@ -24,7 +24,7 @@ static FBlitter_32bppOptimized iFBlitter @@
  * @param bp further blitting parameters
  * @param zoom zoom level at which we are drawing
  */
 template <BlitterMode mode>
+template <BlitterMode mode, bool Tpal_to_rgb>
 inline void Blitter_32bppOptimized::Draw(const Blitter::BlitterParams *bp, ZoomLevel zoom)
+{
 	const SpriteData *src = (const SpriteData *)bp->sprite;
@@ @@ -209,18 +209,29 @@ inline void Blitter_32bppOptimized::Draw @@
 				default:
 					if (src_px->a == 255) {
 						/* faster than memcpy(), n is usually low */
 						src_n += n;
 						do {
 							*dst = src_px->data;
 							if (Tpal_to_rgb && *src_n != 0) {
 								/* Convert the mapping channel to a RGB value */
 								*dst = this->AdjustBrightness(this->LookupColourInPalette(GB(*src_n, 0, 8)), GB(*src_n, 8, 8)).data;
 							} else {
 								*dst = src_px->data;
+							}
 							dst++;
 							src_px++;
 							src_n++;
 						} while (--n != 0);
 					} else {
 						src_n += n;
 						do {
 							*dst = ComposeColourRGBANoCheck(src_px->r, src_px->g, src_px->b, src_px->a, *dst);
 							if (Tpal_to_rgb && *src_n != 0) {
 								/* Convert the mapping channel to a RGB value */
 								Colour colour = this->AdjustBrightness(this->LookupColourInPalette(GB(*src_n, 0, 8)), GB(*src_n, 8, 8));
 								*dst = ComposeColourRGBANoCheck(colour.r, colour.g, colour.b, src_px->a, *dst);
 							} else {
 								*dst = ComposeColourRGBANoCheck(src_px->r, src_px->g, src_px->b, src_px->a, *dst);
+							}
 							dst++;
 							src_px++;
 							src_n++;
 						} while (--n != 0);
+					}
 					break;
@@ @@ -233,6 +244,22 @@ inline void Blitter_32bppOptimized::Draw @@
+	}
+}
 template <bool Tpal_to_rgb>
 void Blitter_32bppOptimized::Draw(Blitter::BlitterParams *bp, BlitterMode mode, ZoomLevel zoom)
+{
 	switch (mode) {
 		default: NOT_REACHED();
 		case BM_NORMAL:       Draw<BM_NORMAL, Tpal_to_rgb>(bp, zoom); return;
 		case BM_COLOUR_REMAP: Draw<BM_COLOUR_REMAP, Tpal_to_rgb>(bp, zoom); return;
 		case BM_TRANSPARENT:  Draw<BM_TRANSPARENT, Tpal_to_rgb>(bp, zoom); return;
 		case BM_CRASH_REMAP:  Draw<BM_CRASH_REMAP, Tpal_to_rgb>(bp, zoom); return;
 		case BM_BLACK_REMAP:  Draw<BM_BLACK_REMAP, Tpal_to_rgb>(bp, zoom); return;
+	}
+}
 template void Blitter_32bppOptimized::Draw<true>(Blitter::BlitterParams *bp, BlitterMode mode, ZoomLevel zoom);
 template void Blitter_32bppOptimized::Draw<false>(Blitter::BlitterParams *bp, BlitterMode mode, ZoomLevel zoom);
 /**
  * Draws a sprite to a (screen) buffer. Calls adequate templated function.
+ *
@@ @@ -242,14 +269,7 @@ inline void Blitter_32bppOptimized::Draw @@
  */
 void Blitter_32bppOptimized::Draw(Blitter::BlitterParams *bp, BlitterMode mode, ZoomLevel zoom)
+{
 	switch (mode) {
 		default: NOT_REACHED();
 		case BM_NORMAL:       Draw<BM_NORMAL>      (bp, zoom); return;
 		case BM_COLOUR_REMAP: Draw<BM_COLOUR_REMAP>(bp, zoom); return;
 		case BM_TRANSPARENT:  Draw<BM_TRANSPARENT> (bp, zoom); return;
 		case BM_CRASH_REMAP:  Draw<BM_CRASH_REMAP> (bp, zoom); return;
 		case BM_BLACK_REMAP:  Draw<BM_BLACK_REMAP> (bp, zoom); return;
+	}
 	this->Draw<false>(bp, mode, zoom);
+}
 template <bool Tpal_to_rgb> Sprite *Blitter_32bppOptimized::EncodeInternal(const SpriteLoader::Sprite *sprite, AllocatorProc *allocator)
@@ @@ -322,19 +342,21 @@ template <bool Tpal_to_rgb> Sprite *Blit @@
 				if (a != 0) {
 					dst_px->a = a;
 					*dst_n = src->m;
-					if (Tpal_to_rgb && src->m != 0) {
 					if (src->m != 0) {
 						/* Get brightest value */
 						uint8 rgb_max = std::max({src->r, src->g, src->b});
+						uint8 rgb_max = std::max({ src->r, src->g, src->b });
 						/* Black pixel (8bpp or old 32bpp image), so use default value */
 						if (rgb_max == 0) rgb_max = DEFAULT_BRIGHTNESS;
 						*dst_n |= rgb_max << 8;
 						/* Pre-convert the mapping channel to a RGB value */
 						Colour colour = this->AdjustBrightness(this->LookupColourInPalette(src->m), rgb_max);
 						dst_px->r = colour.r;
 						dst_px->g = colour.g;
 						dst_px->b = colour.b;
 						if (Tpal_to_rgb) {
 							/* Pre-convert the mapping channel to a RGB value */
 							Colour colour = this->AdjustBrightness(this->LookupColourInPalette(src->m), rgb_max);
 							dst_px->r = colour.r;
 							dst_px->g = colour.g;
 							dst_px->b = colour.b;
+						}
 					} else {
 						dst_px->r = src->r;
 						dst_px->g = src->g;

src/blitter/32bpp_optimized.hpp

➞

Show inline comments

@@ @@ -26,9 +26,10 @@ public: @@
 	const char *GetName() override { return "32bpp-optimized"; }
 	template <BlitterMode mode> void Draw(const Blitter::BlitterParams *bp, ZoomLevel zoom);
+	template <BlitterMode mode, bool Tpal_to_rgb = false> void Draw(const Blitter::BlitterParams *bp, ZoomLevel zoom);
 protected:
 	template <bool Tpal_to_rgb> void Draw(Blitter::BlitterParams *bp, BlitterMode mode, ZoomLevel zoom);
 	template <bool Tpal_to_rgb> Sprite *EncodeInternal(const SpriteLoader::Sprite *sprite, AllocatorProc *allocator);
 };

src/blitter/40bpp_anim.cpp

➞

Show inline comments

@@ @@ -315,7 +315,7 @@ void Blitter_40bppAnim::Draw(Blitter::Bl @@
 	if (_screen_disable_anim || VideoDriver::GetInstance()->GetAnimBuffer() == nullptr) {
 		/* This means our output is not to the screen, so we can't be doing any animation stuff, so use our parent Draw() */
 		Blitter_32bppOptimized::Draw(bp, mode, zoom);
+		Blitter_32bppOptimized::Draw<true>(bp, mode, zoom);
 		return;
+	}

0 comments (0 inline, 0 general)