Possible error in LEDColor

[MysDevLop]

Walking through the execution path in setting an LED color, there is this method in LedStrip

  public void setPixelColor( int pixel, int color )
    {
        ledColors[pixel] = LedColor.scaleColorToBrightness( color, maxBrightness );
    }

Following into the scale brightness method in LedColor is this method:

  public static int scaleColorToBrightness( int color, double brightness )
        {
            color  = LedColor.setRedComponent( color, (int) (LedColor.getRedComponent( color ) * brightness) );
            color = LedColor.setGreenComponent( color, (int) (LedColor.getGreenComponent( color ) * brightness) );
            color = LedColor.setBlueComponent( color, (int) (LedColor.getBlueComponent( color ) * brightness) );

            return color;
        }

To me, color will always be the value of the blue component.

I think the method should OR the return values, and so contain all the colors:

  public static int scaleColorToBrightness( int color, double brightness )
        {
            color  = LedColor.setRedComponent( color, (int) (LedColor.getRedComponent( color ) * brightness) );
            color |= LedColor.setGreenComponent( color, (int) (LedColor.getGreenComponent( color ) * brightness) );
            color |= LedColor.setBlueComponent( color, (int) (LedColor.getBlueComponent( color ) * brightness) );

            return color;
        }

Thoughts?

[MysDevLop]

I also modified setPixelColor() in LedStrip to make it slightly more efficient, by adding test for full brightness.

  public void setPixelColor( int pixel, int color )
    {
        if (maxBrightness == 1.0d)
            ledColors[pixel] = color;
        else
            ledColors[pixel] = LedColor.scaleColorToBrightness( color, maxBrightness );
    }