From b9c3f8db4fddf294dc0cf178852611813c0c559a Mon Sep 17 00:00:00 2001
From: Randy Heit <rheit@zdoom.fake>
Date: Fri, 11 May 2012 02:05:24 +0000
Subject: [PATCH] - Added a new patch style: CopyNewAlpha. This works just like
 Copy except it multiplies each   pixel's alpha channel by the specified Alpha
 property.

SVN r3637 (trunk)
---
 src/textures/bitmap.cpp            | 111 ++++++-----------------------
 src/textures/bitmap.h              |  27 ++++---
 src/textures/multipatchtexture.cpp |   2 +-
 3 files changed, 41 insertions(+), 99 deletions(-)

diff --git a/src/textures/bitmap.cpp b/src/textures/bitmap.cpp
index 4da12b9c35..cf419d70da 100644
--- a/src/textures/bitmap.cpp
+++ b/src/textures/bitmap.cpp
@@ -185,96 +185,30 @@ void iCopyColors(BYTE *pout, const BYTE *pin, int count, int step, FCopyInfo *in
 
 typedef void (*CopyFunc)(BYTE *pout, const BYTE *pin, int count, int step, FCopyInfo *inf);
 
+#define COPY_FUNCS(op) \
+	{ \
+		iCopyColors<cRGB, cBGRA, op>, \
+		iCopyColors<cRGBA, cBGRA, op>, \
+		iCopyColors<cIA, cBGRA, op>, \
+		iCopyColors<cCMYK, cBGRA, op>, \
+		iCopyColors<cBGR, cBGRA, op>, \
+		iCopyColors<cBGRA, cBGRA, op>, \
+		iCopyColors<cI16, cBGRA, op>, \
+		iCopyColors<cRGB555, cBGRA, op>, \
+		iCopyColors<cPalEntry, cBGRA, op> \
+	}
 static const CopyFunc copyfuncs[][9]={
-	{
-		iCopyColors<cRGB, cBGRA, bCopy>,
-		iCopyColors<cRGBA, cBGRA, bCopy>,
-		iCopyColors<cIA, cBGRA, bCopy>,
-		iCopyColors<cCMYK, cBGRA, bCopy>,
-		iCopyColors<cBGR, cBGRA, bCopy>,
-		iCopyColors<cBGRA, cBGRA, bCopy>,
-		iCopyColors<cI16, cBGRA, bCopy>,
-		iCopyColors<cRGB555, cBGRA, bCopy>,
-		iCopyColors<cPalEntry, cBGRA, bCopy>
-	},
-	{
-		iCopyColors<cRGB, cBGRA, bBlend>,
-		iCopyColors<cRGBA, cBGRA, bBlend>,
-		iCopyColors<cIA, cBGRA, bBlend>,
-		iCopyColors<cCMYK, cBGRA, bBlend>,
-		iCopyColors<cBGR, cBGRA, bBlend>,
-		iCopyColors<cBGRA, cBGRA, bBlend>,
-		iCopyColors<cI16, cBGRA, bBlend>,
-		iCopyColors<cRGB555, cBGRA, bBlend>,
-		iCopyColors<cPalEntry, cBGRA, bBlend>
-	},
-	{
-		iCopyColors<cRGB, cBGRA, bAdd>,
-		iCopyColors<cRGBA, cBGRA, bAdd>,
-		iCopyColors<cIA, cBGRA, bAdd>,
-		iCopyColors<cCMYK, cBGRA, bAdd>,
-		iCopyColors<cBGR, cBGRA, bAdd>,
-		iCopyColors<cBGRA, cBGRA, bAdd>,
-		iCopyColors<cI16, cBGRA, bAdd>,
-		iCopyColors<cRGB555, cBGRA, bAdd>,
-		iCopyColors<cPalEntry, cBGRA, bAdd>
-	},
-	{
-		iCopyColors<cRGB, cBGRA, bSubtract>,
-		iCopyColors<cRGBA, cBGRA, bSubtract>,
-		iCopyColors<cIA, cBGRA, bSubtract>,
-		iCopyColors<cCMYK, cBGRA, bSubtract>,
-		iCopyColors<cBGR, cBGRA, bSubtract>,
-		iCopyColors<cBGRA, cBGRA, bSubtract>,
-		iCopyColors<cI16, cBGRA, bSubtract>,
-		iCopyColors<cRGB555, cBGRA, bSubtract>,
-		iCopyColors<cPalEntry, cBGRA, bSubtract>
-	},
-	{
-		iCopyColors<cRGB, cBGRA, bReverseSubtract>,
-		iCopyColors<cRGBA, cBGRA, bReverseSubtract>,
-		iCopyColors<cIA, cBGRA, bReverseSubtract>,
-		iCopyColors<cCMYK, cBGRA, bReverseSubtract>,
-		iCopyColors<cBGR, cBGRA, bReverseSubtract>,
-		iCopyColors<cBGRA, cBGRA, bReverseSubtract>,
-		iCopyColors<cI16, cBGRA, bReverseSubtract>,
-		iCopyColors<cRGB555, cBGRA, bReverseSubtract>,
-		iCopyColors<cPalEntry, cBGRA, bReverseSubtract>
-	},
-	{
-		iCopyColors<cRGB, cBGRA, bModulate>,
-		iCopyColors<cRGBA, cBGRA, bModulate>,
-		iCopyColors<cIA, cBGRA, bModulate>,
-		iCopyColors<cCMYK, cBGRA, bModulate>,
-		iCopyColors<cBGR, cBGRA, bModulate>,
-		iCopyColors<cBGRA, cBGRA, bModulate>,
-		iCopyColors<cI16, cBGRA, bModulate>,
-		iCopyColors<cRGB555, cBGRA, bModulate>,
-		iCopyColors<cPalEntry, cBGRA, bModulate>
-	},
-	{
-		iCopyColors<cRGB, cBGRA, bCopyAlpha>,
-		iCopyColors<cRGBA, cBGRA, bCopyAlpha>,
-		iCopyColors<cIA, cBGRA, bCopyAlpha>,
-		iCopyColors<cCMYK, cBGRA, bCopyAlpha>,
-		iCopyColors<cBGR, cBGRA, bCopyAlpha>,
-		iCopyColors<cBGRA, cBGRA, bCopyAlpha>,
-		iCopyColors<cI16, cBGRA, bCopyAlpha>,
-		iCopyColors<cRGB555, cBGRA, bCopyAlpha>,
-		iCopyColors<cPalEntry, cBGRA, bCopyAlpha>
-	},
-	{
-		iCopyColors<cRGB, cBGRA, bOverwrite>,
-		iCopyColors<cRGBA, cBGRA, bOverwrite>,
-		iCopyColors<cIA, cBGRA, bOverwrite>,
-		iCopyColors<cCMYK, cBGRA, bOverwrite>,
-		iCopyColors<cBGR, cBGRA, bOverwrite>,
-		iCopyColors<cBGRA, cBGRA, bOverwrite>,
-		iCopyColors<cI16, cBGRA, bOverwrite>,
-		iCopyColors<cRGB555, cBGRA, bOverwrite>,
-		iCopyColors<cPalEntry, cBGRA, bOverwrite>
-	},
+	COPY_FUNCS(bCopy),
+	COPY_FUNCS(bBlend),
+	COPY_FUNCS(bAdd),
+	COPY_FUNCS(bSubtract),
+	COPY_FUNCS(bReverseSubtract),
+	COPY_FUNCS(bModulate),
+	COPY_FUNCS(bCopyAlpha),
+	COPY_FUNCS(bCopyNewAlpha),
+	COPY_FUNCS(bOverwrite)
 };
+#undef COPY_FUNCS
 
 //===========================================================================
 //
@@ -487,6 +421,7 @@ static const CopyPalettedFunc copypalettedfuncs[]=
 	iCopyPaletted<cBGRA, bReverseSubtract>,
 	iCopyPaletted<cBGRA, bModulate>,
 	iCopyPaletted<cBGRA, bCopyAlpha>,
+	iCopyPaletted<cBGRA, bCopyNewAlpha>,
 	iCopyPaletted<cBGRA, bOverwrite>
 };
 
diff --git a/src/textures/bitmap.h b/src/textures/bitmap.h
index da039fd0f8..dc125bf87e 100644
--- a/src/textures/bitmap.h
+++ b/src/textures/bitmap.h
@@ -318,6 +318,7 @@ enum ECopyOp
 	OP_REVERSESUBTRACT,
 	OP_MODULATE,
 	OP_COPYALPHA,
+	OP_COPYNEWALPHA,
 	OP_OVERWRITE
 };
 
@@ -330,6 +331,13 @@ struct FCopyInfo
 	fixed_t invalpha;
 };
 
+struct bOverwrite
+{
+	static __forceinline void OpC(BYTE &d, BYTE s, BYTE a, FCopyInfo *i) { d = s; }
+	static __forceinline void OpA(BYTE &d, BYTE s, FCopyInfo *i) { d = s; }
+	static __forceinline bool ProcessAlpha0() { return true; }
+};
+
 struct bCopy
 {
 	static __forceinline void OpC(BYTE &d, BYTE s, BYTE a, FCopyInfo *i) { d = s; }
@@ -337,11 +345,18 @@ struct bCopy
 	static __forceinline bool ProcessAlpha0() { return false; }
 };
 
-struct bOverwrite
+struct bCopyNewAlpha
 {
 	static __forceinline void OpC(BYTE &d, BYTE s, BYTE a, FCopyInfo *i) { d = s; }
+	static __forceinline void OpA(BYTE &d, BYTE s, FCopyInfo *i) { d = (s*i->alpha) >> FRACBITS; }
+	static __forceinline bool ProcessAlpha0() { return false; }
+};
+
+struct bCopyAlpha
+{
+	static __forceinline void OpC(BYTE &d, BYTE s, BYTE a, FCopyInfo *i) { d = (s*a + d*(255-a))/255; }
 	static __forceinline void OpA(BYTE &d, BYTE s, FCopyInfo *i) { d = s; }
-	static __forceinline bool ProcessAlpha0() { return true; }
+	static __forceinline bool ProcessAlpha0() { return false; }
 };
 
 struct bBlend
@@ -379,13 +394,5 @@ struct bModulate
 	static __forceinline bool ProcessAlpha0() { return false; }
 };
 
-struct bCopyAlpha
-{
-	static __forceinline void OpC(BYTE &d, BYTE s, BYTE a, FCopyInfo *i) { d = (s*a + d*(255-a))/255; }
-	static __forceinline void OpA(BYTE &d, BYTE s, FCopyInfo *i) { d = s; }
-	static __forceinline bool ProcessAlpha0() { return false; }
-};
-
-
 
 #endif
diff --git a/src/textures/multipatchtexture.cpp b/src/textures/multipatchtexture.cpp
index c1fc95492a..5799ad04ea 100644
--- a/src/textures/multipatchtexture.cpp
+++ b/src/textures/multipatchtexture.cpp
@@ -1180,7 +1180,7 @@ void FMultiPatchTexture::ParsePatch(FScanner &sc, TexPart & part, bool silent, i
 			}
 			else if (sc.Compare("style"))
 			{
-				static const char *styles[] = {"copy", "translucent", "add", "subtract", "reversesubtract", "modulate", "copyalpha", NULL };
+				static const char *styles[] = {"copy", "translucent", "add", "subtract", "reversesubtract", "modulate", "copyalpha", "copynewalpha", NULL };
 				sc.MustGetString();
 				part.op = sc.MustMatchString(styles);
 				bComplex |= (part.op != OP_COPY);