X-Git-Url: https://git.sesse.net/?p=movit;a=blobdiff_plain;f=padding_effect.cpp;h=40876b89e9a6edca0cf2c89146894c1d72f15070;hp=3c4919a0c3b1b98a3aa31ebf9aad14ff536bcdcf;hb=af82a6c84f57851d16939b86188afb5fb80819d0;hpb=5d4c0425579de66b3e2dd7d0095e890278f8efcf

diff --git a/padding_effect.cpp b/padding_effect.cpp
index 3c4919a..40876b8 100644
--- a/padding_effect.cpp
+++ b/padding_effect.cpp
@@ -1,6 +1,7 @@
-#include <math.h>
 #include <GL/glew.h>
+#include <assert.h>
 
+#include "effect_util.h"
 #include "padding_effect.h"
 #include "util.h"
 
@@ -89,13 +90,25 @@ bool PaddingEffect::needs_srgb_primaries() const
 	return true;
 }
 
-// If the border color is black, it doesn't matter if we're pre- or postmultiplied
-// (or even blank, as a hack). Otherwise, it does.
 Effect::AlphaHandling PaddingEffect::alpha_handling() const
 {
-	if (border_color.r == 0.0 && border_color.g == 0.0 && border_color.b == 0.0) {
+	// If the border color is black, it doesn't matter if we're pre- or postmultiplied.
+	// Note that for non-solid black (i.e. alpha < 1.0), we're equally fine with
+	// pre- and postmultiplied, but later effects might change this status
+	// (consider e.g. blur), so setting DONT_CARE_ALPHA_TYPE is inappropriate,
+	// as it propagate blank alpha through this effect.
+	if (border_color.r == 0.0 && border_color.g == 0.0 && border_color.b == 0.0 && border_color.a == 1.0) {
 		return DONT_CARE_ALPHA_TYPE;
 	}
+
+	// If the border color is solid, we preserve blank alpha, as we never output any
+	// new non-solid pixels.
+	if (border_color.a == 1.0) {
+		return INPUT_PREMULTIPLIED_ALPHA_KEEP_BLANK;
+	}
+
+	// Otherwise, we're going to output our border color in premultiplied alpha,
+	// so the other pixels better be premultiplied as well.
 	return INPUT_AND_OUTPUT_PREMULTIPLIED_ALPHA;
 }