From fbac6311601ac636bc594d5f6cede173d3972e37 Mon Sep 17 00:00:00 2001
From: "ShA..Rd" <ShAdOwIsLoRd@gmail.com>
Date: Fri, 19 Apr 2013 16:58:40 +0000
Subject: [PATCH]  * TextureArray constructor ensures all images passed to it
 have the same format and dimensions, otherwise an IllegalArgumentException is
 thrown

git-svn-id: https://jmonkeyengine.googlecode.com/svn/trunk@10562 75d07b2b-3a1a-0410-a2c5-0572b91ccdca
---
 .../core/com/jme3/texture/TextureArray.java   | 20 ++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

diff --git a/engine/src/core/com/jme3/texture/TextureArray.java b/engine/src/core/com/jme3/texture/TextureArray.java
index 2bdc89a30..32cf376dd 100644
--- a/engine/src/core/com/jme3/texture/TextureArray.java
+++ b/engine/src/core/com/jme3/texture/TextureArray.java
@@ -31,6 +31,7 @@
  */
 package com.jme3.texture;
 
+import com.jme3.texture.Image.Format;
 import java.util.List;
 import java.util.logging.Level;
 import java.util.logging.Logger;
@@ -58,26 +59,27 @@ public class TextureArray extends Texture {
     }
 
     /**
-     * Construct a TextureArray from the given list of images
-     * warning, this feature is only supported on opengl 3.0 version.
+     * Construct a TextureArray from the given list of images.
      * To check if a hardware supports TextureArray check : 
      * renderManager.getRenderer().getCaps().contains(Caps.TextureArray)
      * @param images 
      */
     public TextureArray(List<Image> images) {
-        super();        
+        super();
+        
         int width = images.get(0).getWidth();
         int height = images.get(0).getHeight();
-        Image arrayImage = new Image(images.get(0).getFormat(), width, height,
-                null);
+        Format format = images.get(0).getFormat();
+        Image arrayImage = new Image(format, width, height, null);
 
         for (Image img : images) {
             if (img.getHeight() != height || img.getWidth() != width) {
-                Logger.getLogger(TextureArray.class.getName()).log(
-                        Level.WARNING,
-                        "all images must have the same width/height");
-                continue;
+                throw new IllegalArgumentException("Images in texture array must have same dimensions");
+            }
+            if (img.getFormat() != format) {
+                throw new IllegalArgumentException("Images in texture array must have same format");
             }
+            
             arrayImage.addData(img.getData(0));
         }
         setImage(arrayImage);