Re: Patch for java.awt.image.ColorModel

[Thomas Fitzsimmons]

On Mon, 2004-04-05 at 07:11, Ingo Prötel wrote:
> I suggest the following patch
> 
> 2004-04-05  Ingo Proetel <address@hidden>
> 
>       * java/awt/image/ColorModel.java (getRGBdefault): Default ColorModel 
> has 32 bit pixels not 8 bit 
> pixels.
>       (isCompatibleRaster): Added javadoc comment.
> 
> ingo


> -  // Typically overridden
> +  /**
> +   * Checks if the given raster has a compatible data-layout
> (SampleModel).
> +   * @param raster The Raster to test.
> +   * @return true if raster is compatible.
> +   */ 

This implementation is different from the description in Sun's
javadocs.  Their default implementation is to throw an
UnsupportedOperationException, since ColorModel is an abstract class. 
Does the implementation here need to be moved into one of the derived
classes?

>    public boolean isCompatibleRaster(Raster raster)
>    {
>      SampleModel sampleModel = raster.getSampleModel();

Tom