Performance improvements

jme3-android/src/main/java/com/jme3/input/android/AndroidJoyInput.java

@@ -215,7 +215,7 @@ public Joystick[] loadJoysticks(InputManager inputManager) {
         if (!disableSensors) {
             joystickList.add(sensorJoyInput.loadJoystick(joystickList.size(), inputManager));
         }
-        return joystickList.toArray( new Joystick[joystickList.size()] );
+        return joystickList.toArray(new Joystick[0]);
     }
 
     @Override

jme3-android/src/main/java/com/jme3/input/android/AndroidJoyInput14.java

@@ -94,7 +94,7 @@ public Joystick[] loadJoysticks(InputManager inputManager) {
         // load physical gamepads/joysticks
         joystickList.addAll(joystickJoyInput.loadJoysticks(joystickList.size(), inputManager));
         // return the list of joysticks back to InputManager
-        return joystickList.toArray( new Joystick[joystickList.size()] );
+        return joystickList.toArray(new Joystick[0]);
     }
 
     public boolean onGenericMotion(MotionEvent event) {

jme3-android/src/main/java/com/jme3/input/android/AndroidSensorJoyInput.java

@@ -546,9 +546,8 @@ public void onSensorChanged(SensorEvent se) {
                 return;
             }
             synchronized(sensorData.valuesLock) {
-                for (int i=0; i<sensorData.lastValues.length; i++) {
-                    sensorData.lastValues[i] = se.values[i];
-                }
+                if (sensorData.lastValues.length >= 0)
+                    System.arraycopy(se.values, 0, sensorData.lastValues, 0, sensorData.lastValues.length);
             }
 
             if (sensorData.axes.size() > 0) {

jme3-core/src/main/java/com/jme3/anim/Armature.java

@@ -83,7 +83,7 @@ public Armature(Joint[] jointList) {
                 rootJointList.add(joint);
             }
         }
-        rootJoints = rootJointList.toArray(new Joint[rootJointList.size()]);
+        rootJoints = rootJointList.toArray(new Joint[0]);
 
         createSkinningMatrices();
 

jme3-core/src/main/java/com/jme3/anim/tween/Tweens.java

@@ -597,13 +597,9 @@ public CallTweenMethod(double length, Object target, String methodName, Object..
             // So now set up the real args list.
             this.args = new Object[args.length + 1];
             if (tIndex == 0) {
-                for (int i = 0; i < args.length; i++) {
-                    this.args[i + 1] = args[i];
-                }
+                System.arraycopy(args, 0, this.args, 1, args.length);
             } else {
-                for (int i = 0; i < args.length; i++) {
-                    this.args[i] = args[i];
-                }
+                System.arraycopy(args, 0, this.args, 0, args.length);
             }
         }
 

jme3-core/src/main/java/com/jme3/anim/util/AnimMigrationUtils.java

@@ -94,7 +94,7 @@ public void visit(Spatial spatial) {
                         padJointTracks(tracks, staticJoints[i]);
                     }
 
-                    clip.setTracks(tracks.toArray(new TransformTrack[tracks.size()]));
+                    clip.setTracks(tracks.toArray(new TransformTrack[0]));
 
                     composer.addAnimClip(clip);
                 }

jme3-core/src/main/java/com/jme3/animation/Animation.java

@@ -39,6 +39,7 @@
 import com.jme3.util.clone.JmeCloneable;
 
 import java.io.IOException;
+import java.util.Collections;
 
 /**
  * The animation class updates the animation target with the tracks of a given type.
@@ -139,9 +140,7 @@ void setTime(float time, float blendAmount, AnimControl control, AnimChannel cha
      * @param tracksArray The tracks to set.
      */
     public void setTracks(Track[] tracksArray) {
-        for (Track track : tracksArray) {
-            tracks.add(track);
-        }
+        Collections.addAll(tracks, tracksArray);
     }
 
     /**

jme3-core/src/main/java/com/jme3/animation/Skeleton.java

@@ -81,7 +81,7 @@ public Skeleton(Bone[] boneList) {
                 rootBoneList.add(b);
             }
         }
-        rootBones = rootBoneList.toArray(new Bone[rootBoneList.size()]);
+        rootBones = rootBoneList.toArray(new Bone[0]);
 
         createSkinningMatrices();
 

jme3-core/src/main/java/com/jme3/app/DetailedProfiler.java

@@ -161,8 +161,7 @@ public void vpStep(VpStep step, ViewPort vp, RenderQueue.Bucket bucket) {
                 curVpPath = vpPath.toString();
             } else {
                 if (bucket != null) {
-                    path.append(curAppPath).append("/").append(curVpPath).append("/")
-                        .append(bucket.name() + " Bucket");
+                    path.append(curAppPath).append("/").append(curVpPath).append("/").append(bucket.name()).append(" Bucket");
                 } else {
                     path.append(curAppPath).append("/").append(vpPath);
                     curVpPath = vpPath.toString();

jme3-core/src/main/java/com/jme3/app/state/ScreenshotAppState.java

@@ -51,6 +51,7 @@
 import java.io.IOException;
 import java.io.OutputStream;
 import java.nio.ByteBuffer;
+import java.nio.file.Files;
 import java.util.List;
 import java.util.logging.Level;
 import java.util.logging.Logger;
@@ -326,11 +327,8 @@ public void setProfiler(AppProfiler profiler) {
      * @throws IOException if an I/O error occurs
      */
     protected void writeImageFile(File file) throws IOException {
-        OutputStream outStream = new FileOutputStream(file);
-        try {
+        try(OutputStream outStream = Files.newOutputStream(file.toPath())) {
             JmeSystem.writeImageFile(outStream, "png", outBuf, width, height);
-        } finally {
-            outStream.close();
         }
     }
 }

jme3-core/src/main/java/com/jme3/cinematic/Cinematic.java

@@ -221,7 +221,7 @@ public void onPause() {
     public void write(JmeExporter ex) throws IOException {
         super.write(ex);
         OutputCapsule oc = ex.getCapsule(this);
-        oc.write(cinematicEvents.toArray(new CinematicEvent[cinematicEvents.size()]), "cinematicEvents", null);
+        oc.write(cinematicEvents.toArray(new CinematicEvent[0]), "cinematicEvents", null);
         oc.writeStringSavableMap(cameras, "cameras", null);
         oc.write(timeLine, "timeLine", null);
 

jme3-core/src/main/java/com/jme3/cinematic/TimeLine.java

@@ -106,8 +106,7 @@ public int getLastKeyFrameIndex() {
     @SuppressWarnings("unchecked")
     public void write(JmeExporter ex) throws IOException {
         OutputCapsule oc = ex.getCapsule(this);
-        ArrayList list = new ArrayList();
-        list.addAll(values());
+        ArrayList list = new ArrayList(values());
         oc.writeSavableArrayList(list, "keyFrames", null);
     }
 

jme3-core/src/main/java/com/jme3/light/LightList.java

@@ -334,10 +334,7 @@ public void write(JmeExporter ex) throws IOException {
         OutputCapsule oc = ex.getCapsule(this);
 //        oc.write(owner, "owner", null);
 
-        ArrayList<Light> lights = new ArrayList<>();
-        for (int i = 0; i < listSize; i++) {
-            lights.add(list[i]);
-        }
+        ArrayList<Light> lights = new ArrayList<>(Arrays.asList(list).subList(0, listSize));
         oc.writeSavableArrayList(lights, "lights", null);
     }
 

jme3-core/src/main/java/com/jme3/math/Spline.java

@@ -34,6 +34,7 @@
 import com.jme3.export.*;
 import java.io.IOException;
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.Iterator;
 import java.util.List;
 
@@ -84,10 +85,7 @@ public Spline(SplineType splineType, Vector3f[] controlPoints, float curveTensio
         if (splineType == SplineType.Nurb) {
             throw new IllegalArgumentException("To create NURBS spline use: 'public Spline(Vector3f[] controlPoints, float[] weights, float[] nurbKnots)' constructor!");
         }
-        for (int i = 0; i < controlPoints.length; i++) {
-            Vector3f vector3f = controlPoints[i];
-            this.controlPoints.add(vector3f);
-        }
+        Collections.addAll(this.controlPoints, controlPoints);
         type = splineType;
         this.curveTension = curveTension;
         this.cycle = cycle;
@@ -164,10 +162,7 @@ private void initCatmullRomWayPoints(List<Vector3f> list) {
             CRcontrolPoints.add(list.get(0).subtract(list.get(1).subtract(list.get(0))));
         }
 
-        for (Iterator<Vector3f> it = list.iterator(); it.hasNext();) {
-            Vector3f vector3f = it.next();
-            CRcontrolPoints.add(vector3f);
-        }
+        CRcontrolPoints.addAll(list);
         if (cycle) {
             CRcontrolPoints.add(list.get(1));
         } else {

jme3-core/src/main/java/com/jme3/opencl/DefaultPlatformChooser.java

@@ -77,9 +77,7 @@ public List<? extends Device> chooseDevices(List<? extends Platform> platforms)
         //still no one found, try without interop
         LOG.warning("No device with OpenCL-OpenGL-interop found, try without");
         for (Platform p : platforms) {
-            for (Device d : p.getDevices()) {
-                result.add(d);
-            }
+            result.addAll(p.getDevices());
             if (!result.isEmpty()) {
                 return result;
             }

jme3-core/src/main/java/com/jme3/renderer/opengl/GLRenderer.java

@@ -1398,7 +1398,7 @@ assert isValidNumber(fb) : "Invalid Matrix4f value " + uniform.getValue() + " fo
             case FloatArray:
                 fb = uniform.getMultiData();
                 assert isValidNumber(fb) : "Invalid float array value "
-                        + Arrays.asList((float[]) uniform.getValue()) + " for " + uniform.getBinding();
+                        + Arrays.toString((float[]) uniform.getValue()) + " for " + uniform.getBinding();
                 gl.glUniform1(loc, fb);
                 break;
             case Vector2Array:

jme3-core/src/main/java/com/jme3/scene/Mesh.java

@@ -705,8 +705,7 @@ public void setStreamed() {
      */
     @Deprecated
     public void setInterleaved() {
-        ArrayList<VertexBuffer> vbs = new ArrayList<>();
-        vbs.addAll(buffersList);
+        ArrayList<VertexBuffer> vbs = new ArrayList<>(buffersList);
 
 //        ArrayList<VertexBuffer> vbs = new ArrayList<VertexBuffer>(buffers.values());
         // index buffer not included when interleaving

jme3-core/src/main/java/com/jme3/scene/instancing/InstancedGeometry.java

@@ -420,7 +420,7 @@ private void updateAllInstanceData() {
         if (globalInstanceData != null) {
             allData.addAll(Arrays.asList(globalInstanceData));
         }
-        allInstanceData = allData.toArray(new VertexBuffer[allData.size()]);
+        allInstanceData = allData.toArray(new VertexBuffer[0]);
     }
 
     @Override

jme3-core/src/main/java/com/jme3/scene/shape/Surface.java

@@ -43,8 +43,8 @@
 import com.jme3.scene.Mesh;
 import com.jme3.scene.VertexBuffer;
 import com.jme3.util.BufferUtils;
-import java.io.IOException;
 
+import java.io.IOException;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
@@ -229,7 +229,7 @@ private void buildSurface(boolean smooth) {
             }
         }
 
-        Vector3f[] verticesArray = vertices.toArray(new Vector3f[vertices.size()]);
+        Vector3f[] verticesArray = vertices.toArray(new Vector3f[0]);
         // normalMap merges normals of faces that will be rendered smooth
         Map<Vector3f, Vector3f> normalMap = new HashMap<>(verticesArray.length);
         for (int i = 0; i < indices.length; i += 3) {

jme3-core/src/main/java/com/jme3/shader/Glsl100ShaderGenerator.java

@@ -506,6 +506,7 @@ protected boolean isVarying(ShaderGenerationInfo info, ShaderNodeVariable v) {
         for (ShaderNodeVariable shaderNodeVariable : info.getVaryings()) {
             if (shaderNodeVariable.equals(v)) {
                 isVarying = true;
+                break;
             }
         }
         return isVarying;

jme3-core/src/main/java/com/jme3/util/JmeFormatter.java

@@ -52,7 +52,7 @@ public class JmeFormatter extends Formatter {
     final private StringBuffer store = new StringBuffer();
 
     public JmeFormatter(){
-        lineSeparator = System.getProperty("line.separator");
+        lineSeparator = System.lineSeparator();
         format = new MessageFormat("{0,time}");
     }
 

jme3-core/src/main/java/com/jme3/util/TangentBinormalGenerator.java

@@ -113,9 +113,7 @@ public TriangleData(Vector3f tangent, Vector3f binormal, Vector3f normal) {
         }
 
         public void setIndex(int[] index) {
-            for (int i = 0; i < index.length; i++) {
-                this.index[i] = index[i];
-            }
+            System.arraycopy(index, 0, this.index, 0, index.length);
         }
     }
 

jme3-core/src/main/java/com/jme3/util/xml/SAXUtil.java

@@ -105,7 +105,7 @@ public static float parseFloat(String f) throws SAXException {
     }
 
     public static boolean parseBool(String bool, boolean def) throws SAXException {
-        if (bool == null || bool.equals(""))
+        if (bool == null || bool.isEmpty())
             return def;
         else
             return Boolean.valueOf(bool);

jme3-core/src/plugins/java/com/jme3/export/binary/BinaryExporter.java

@@ -39,6 +39,7 @@
 import com.jme3.export.SavableClassUtil;
 import com.jme3.math.FastMath;
 import java.io.*;
+import java.nio.file.Files;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.IdentityHashMap;
@@ -320,8 +321,8 @@ private int findPrevMatch(BinaryIdContentPair oldPair,
     protected byte[] fixClassAlias(byte[] bytes, int width) {
         if (bytes.length != width) {
             byte[] newAlias = new byte[width];
-            for (int x = width - bytes.length; x < width; x++)
-                newAlias[x] = bytes[x - bytes.length];
+            if (bytes.length >= 0)
+                System.arraycopy(bytes, width - 2 * bytes.length, newAlias, width - bytes.length, bytes.length);
             return newAlias;
         }
         return bytes;
@@ -334,8 +335,8 @@ public void save(Savable object, File f, boolean createDirectories) throws IOExc
             parentDirectory.mkdirs();
         }
 
-        try (FileOutputStream fos = new FileOutputStream(f);
-                BufferedOutputStream bos = new BufferedOutputStream(fos)) {
+        try (OutputStream os = Files.newOutputStream(f.toPath());
+                BufferedOutputStream bos = new BufferedOutputStream(os)) {
             save(object, bos);
         }
     }

jme3-core/src/plugins/java/com/jme3/export/binary/BinaryImporter.java

@@ -38,6 +38,7 @@
 import java.io.*;
 import java.net.URL;
 import java.nio.ByteOrder;
+import java.nio.file.Files;
 import java.util.HashMap;
 import java.util.IdentityHashMap;
 import java.util.logging.Level;
@@ -258,9 +259,10 @@ public Savable load(URL f) throws IOException {
     }
 
     public Savable load(URL f, ReadListener listener) throws IOException {
-        InputStream is = f.openStream();
-        Savable rVal = load(is, listener);
-        is.close();
+        Savable rVal;
+        try (InputStream is = f.openStream()) {
+            rVal = load(is, listener);
+        }
         return rVal;
     }
 
@@ -269,18 +271,16 @@ public Savable load(File f) throws IOException {
     }
 
     public Savable load(File f, ReadListener listener) throws IOException {
-        FileInputStream fis = new FileInputStream(f);
-        try {
-            return load(fis, listener);
-        } finally {
-            fis.close();
+        try (InputStream is = Files.newInputStream(f.toPath())) {
+            return load(is, listener);
         }
     }
 
     public Savable load(byte[] data) throws IOException {
-        ByteArrayInputStream bais = new ByteArrayInputStream(data);
-        Savable rVal = load(bais);
-        bais.close();
+        Savable rVal;
+        try (ByteArrayInputStream bais = new ByteArrayInputStream(data)) {
+            rVal = load(bais);
+        }
         return rVal;
     }
 
@@ -291,18 +291,14 @@ public InputCapsule getCapsule(Savable id) {
 
     protected String readString(InputStream f, int length) throws IOException {
         byte[] data = new byte[length];
-        for(int j = 0; j < length; j++) {
-            data[j] = (byte)f.read();
-        }
+        f.read(data, 0, length);
 
         return new String(data);
     }
 
     protected String readString(int length, int offset) throws IOException {
         byte[] data = new byte[length];
-        for(int j = 0; j < length; j++) {
-            data[j] = dataArray[j+offset];
-        }
+        System.arraycopy(dataArray, offset, data, 0, length);
 
         return new String(data);
     }