spatial lods

Author: riccardobl

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

@@ -180,21 +180,19 @@ public void setIgnoreTransform(boolean ignoreTransform) {
      * levels [1, LodLevels + 1] represent the levels set on the mesh
      * with {@link Mesh#setLodLevels(com.jme3.scene.VertexBuffer[]) }.
      *
-     * @param lod The lod level to set
+     * @param lod The lod level to set. If the requested level is outside the
+     * available range, the closest available level is used.
      */
     @Override
     public void setLodLevel(int lod) {
         assert SceneGraphThreadWarden.assertOnCorrectThread(this);
-        if (mesh.getNumLodLevels() == 0) {
-            throw new IllegalStateException("LOD levels are not set on this mesh");
-        }
-
-        if (lod < 0 || lod >= mesh.getNumLodLevels()) {
-            throw new IllegalArgumentException("LOD level is out of range: " + lod);
+        int numLodLevels = mesh.getNumLodLevels();
+        if (numLodLevels == 0) {
+            lodLevel = 0;
+        } else {
+            lodLevel = Math.max(0, Math.min(lod, numLodLevels - 1));
         }
 
-        lodLevel = lod;
-
         if (isGrouped()) {
             groupNode.onMeshChange(this);
         }

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

@@ -40,6 +40,7 @@
 import com.jme3.light.Light;
 import com.jme3.light.LightList;
 import com.jme3.material.Material;
+import com.jme3.scene.control.SpatialLodControl;
 import com.jme3.scene.threadwarden.SceneGraphThreadWarden;
 import com.jme3.util.SafeArrayList;
 import com.jme3.util.clone.Cloner;
@@ -630,6 +631,11 @@ public void setMaterial(Material mat) {
     @Override
     public void setLodLevel(int lod) {
         super.setLodLevel(lod);
+        SpatialLodControl lodControl = getControl(SpatialLodControl.class);
+        if (lodControl != null) {
+            lodControl.setSelectedLodLevel(lod);
+            lodControl.applyLodLevel(lod);
+        }
         for (Spatial child : children.getArray()) {
             child.setLodLevel(lod);
         }
@@ -815,8 +821,19 @@ public void cloneFields(Cloner cloner, Object original) {
     @Override
     @SuppressWarnings("unchecked")
     public void write(JmeExporter e) throws IOException {
-        super.write(e);
-        e.getCapsule(this).writeSavableArrayList(new ArrayList(children), "children", null);
+        SpatialLodControl lodControl = getControl(SpatialLodControl.class);
+        int previousActiveLodLevel = 0;
+        if (lodControl != null) {
+            previousActiveLodLevel = lodControl.prepareForSerialization();
+        }
+        try {
+            super.write(e);
+            e.getCapsule(this).writeSavableArrayList(new ArrayList(children), "children", null);
+        } finally {
+            if (lodControl != null) {
+                lodControl.restoreAfterSerialization(previousActiveLodLevel);
+            }
+        }
     }
 
     @Override

jme3-core/src/main/java/com/jme3/scene/control/LodControl.java

@@ -56,7 +56,11 @@
  * requires the camera to do this. The controlRender method is called each frame
  * and will update the spatial's LOD if the camera has moved by a specified
  * amount.
+ *
+ * @deprecated Use a node-level LOD control for scene-graph LOD switching. This
+ * control only selects pre-existing mesh index-buffer LODs on a Geometry.
  */
+@Deprecated
 public class LodControl extends AbstractControl implements Cloneable, JmeCloneable {
 
     private float trisPerPixel = 1f;