Fix calculation using uninitialized value.
authorDavid Robillard <d@drobilla.net>
Thu, 27 Nov 2014 09:15:30 +0000 (04:15 -0500)
committerDavid Robillard <d@drobilla.net>
Thu, 27 Nov 2014 09:15:30 +0000 (04:15 -0500)
Implicit "this" is the devil.

libs/canvas/colors.cc

index be55d2b7e24de985b53a3d785c449425d1f5ff97..b053e2d1de20a343d87e5229d2d71a57e026e899 100644 (file)
@@ -409,15 +409,15 @@ HSV::distance (const HSV& other) const
 
        double sL, sA, sB;
        double oL, oA, oB;
-       double r, g, b, a;
+       double r, g, b, alpha;  // Careful, "a" is a field of this
        Color c; 
 
        c = hsva_to_color (h, s, v, a);
-       color_to_rgba (c, r, g, b, a);
+       color_to_rgba (c, r, g, b, alpha);
        Rgb2Lab (&sL, &sA, &sB, r, g, b);
 
        c = hsva_to_color (other.h, other.s, other.v, other.a);
-       color_to_rgba (c, r, g, b, a);
+       color_to_rgba (c, r, g, b, alpha);
        Rgb2Lab (&oL, &oA, &oB, r, g, b);
 
        // Weighting factors depending on the application (1 = default)