From 4a2924ab769a4a551a15c78336191117c03d39a8 Mon Sep 17 00:00:00 2001 From: "Steinar H. Gunderson" Date: Mon, 16 Dec 2013 21:25:07 +0100 Subject: [PATCH 1/1] A small comment fix in ColorspaceConversionEffect. --- colorspace_conversion_effect.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/colorspace_conversion_effect.cpp b/colorspace_conversion_effect.cpp index 5406e1b..0e24255 100644 --- a/colorspace_conversion_effect.cpp +++ b/colorspace_conversion_effect.cpp @@ -65,8 +65,8 @@ Matrix3d get_xyz_matrix(Colorspace space) // // Some algebraic fiddling yields (unsurprisingly): // - // X_R = (x_R / y_R) Y_R - // Z_R = (z_R / y_R) Y_R + // X_R = (x_R / y_R) Y_R (so define k1 = x_R / y_R) + // Z_R = (z_R / y_R) Y_R (so define k4 = z_R / y_R) // // We also know that since RGB=(1,1,1) should give us the // D65 illuminant, we must have @@ -75,8 +75,8 @@ Matrix3d get_xyz_matrix(Colorspace space) // Y_R + Y_G + Y_B = D65_Y // Z_R + Z_G + Z_B = D65_Z // - // But since we already know how to express Y and Z by - // some constant multiple of X, this reduces to + // But since we already know how to express X and Z by + // some constant multiple of Y, this reduces to // // k1 Y_R + k2 Y_G + k3 Y_B = D65_X // Y_R + Y_G + Y_B = D65_Y -- 2.39.2