2006-07-17 David Gilbert <david.gilbert@object-refinery.com>

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

2 files changed, 20 insertions, 1 deletions

diff --git a/ChangeLog b/ChangeLog
index 0d02ced70..608e3cec0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2006-07-17  David Gilbert  <david.gilbert@object-refinery.com>
+
+	* java/awt/image/SinglePixelPackedSampleModel.java
+	(createSubsetSampleModel): Added argument check and API docs.
+
 2006-07-17  Roman Kennke  <kennke@aicas.com>
 
 	* gnu/java/awt/peer/x/XGraphics.java
diff --git a/java/awt/image/SinglePixelPackedSampleModel.java b/java/awt/image/SinglePixelPackedSampleModel.java
index 9dd049593..a37fc0bba 100644
--- a/java/awt/image/SinglePixelPackedSampleModel.java
+++ b/java/awt/image/SinglePixelPackedSampleModel.java
@@ -221,9 +221,23 @@ public class SinglePixelPackedSampleModel extends SampleModel
     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;