[cp-patches] FYI: SinglePixelPackedSampleModel - another fix

[David Gilbert]

This patch (committed) fixes another Mauve failure for 
SinglePixelPackedSampleModel:

2006-07-17  David Gilbert  <[EMAIL PROTECTED]>

        * java/awt/image/SinglePixelPackedSampleModel.java
        (createSubsetSampleModel): Added argument check and API docs.

Regards,

Dave

Index: java/awt/image/SinglePixelPackedSampleModel.java
===================================================================
RCS file: 
/sources/classpath/classpath/java/awt/image/SinglePixelPackedSampleModel.java,v
retrieving revision 1.13
diff -u -r1.13 SinglePixelPackedSampleModel.java
--- java/awt/image/SinglePixelPackedSampleModel.java    17 Jul 2006 15:06:28 
-0000      1.13
+++ java/awt/image/SinglePixelPackedSampleModel.java    17 Jul 2006 15:26:09 
-0000
@@ -221,9 +221,23 @@
     return scanlineStride;
   }
 
+  /**
+   * Creates a new <code>SinglePixelPackedSampleModel</code> that accesses
+   * the specified subset of bands.
+   * 
+   * @param bands  an array containing band indices (<code>null</code> not
+   *     permitted).
+   * 
+   * @return A new sample model.
+   * 
+   * @throws NullPointerException if <code>bands</code> is <code>null</code>.
+   * @throws RasterFormatException if <code>bands.length</code> is greater
+   *     than the number of bands in this model.
+   */
   public SampleModel createSubsetSampleModel(int[] bands)
   {
-    // FIXME: Is this the right way to interpret bands?
+    if (bands.length > numBands)
+      throw new RasterFormatException("Too many bands.");
     
     int numBands = bands.length;