Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8255484

All built-in java.awt.color.ColorSpace fields should be specified as such

XMLWordPrintable

    • Icon: CSR CSR
    • Resolution: Approved
    • Icon: P4 P4
    • 16
    • client-libs
    • None
    • 2d
    • minimal
    • Java API
    • SE

      Summary

      The "built-in" wording is inconsistently used for the color space constants in the java.awt.color.ColorSpace class.

      Problem

      For example, the CS_GRAY and CS_LINEAR_RGB spec specifically says that they're "built-in" but not for CS_CIEXYZ and CS_sRGB.

      Solution

      Unify the spec for the ColorSpace.CS_XXX constants.

      Specification

      src/java.desktop/share/classes/java/awt/color/ColorSpace.java 
      
           /**
      -     * The sRGB color space defined at
      +     * The built-in sRGB color space defined at
            * <a href="http://www.w3.org/pub/WWW/Graphics/Color/sRGB.html">
            * http://www.w3.org/pub/WWW/Graphics/Color/sRGB.html</a>.
            */
           @Native public static final int CS_sRGB = 1000;
      
           /**
      -     * A built-in linear RGB color space. This space is based on the same RGB
      +     * The built-in linear RGB color space. This space is based on the same RGB
            * primaries as {@code CS_sRGB}, but has a linear tone reproduction curve.
            */
           @Native public static final int CS_LINEAR_RGB = 1004;
      
           /**
      -     * The CIEXYZ conversion color space defined above.
      +     * The built-in CIEXYZ conversion color space defined above.
            */
           @Native public static final int CS_CIEXYZ = 1001;
      
           /**
      -     * The Photo YCC conversion color space.
      +     * The built-in Photo YCC conversion color space.
            */
           @Native public static final int CS_PYCC = 1002;

            serb Sergey Bylokhov
            rphamsunw Roger Pham (Inactive)
            Philip Race
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: