On Mon, 2004-04-05 at 07:11, Ingo PrÃtel wrote:
> I suggest the following patch
> 
> 2004-04-05  Ingo Proetel <[EMAIL PROTECTED]>
> 
>       * 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




_______________________________________________
Classpath mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/classpath

Reply via email to