Modified EtherealHost to register Vec3d as a serializable class now that

we require it in object state messages.
Modified EtherealHost to pass the client zone extents on to NetworkStateListener.
Modified NetworkStateListener to use the supplied client zone extents.
Upgraded the dependencies to sim-math 1.1.1.

Author: pspeed42

build.gradle

@@ -14,7 +14,7 @@ buildscript {
 apply plugin: 'java'
 apply plugin: 'maven'
 
-version='1.2.1'
+version='1.2.2-SNAPSHOT'
 group='com.simsilica'
 
 ext.jmeVersion = "3.1.0-alpha4" 
@@ -53,7 +53,7 @@ dependencies {
     compile "org.jmonkeyengine:jme3-networking:$jmeVersion" 
 
     // Uses SimMath for bit streaming utilities and transition buffers
-    compile "com.simsilica:sim-math:1.0.2"
+    compile "com.simsilica:sim-math:1.1.1"
 
     // Base logging   
     compile 'org.slf4j:slf4j-api:1.7.15'    

release-notes.md

@@ -1,3 +1,14 @@
+Version 1.2.2 (unreleased)
+--------------
+* Fixed a bug where the newer state messages would fail if the game hadn't
+    already registered Vec3d as a serializable class.
+* Fixed NetworkStateListener to properly pay attention to zone extents.
+* Fixed EtherealHost to pass the specified client zone extents on to the 
+    NetworkStateListener.
+* Added additional trace logging to NetworkStateListener.
+* Upgraded to sim-math 1.1.1   
+
+
 Version 1.2.1
 --------------
 * Added EtherealHost.setStateCollectionInterval() to configure how often

src/main/java/com/simsilica/ethereal/EtherealHost.java

@@ -38,6 +38,7 @@
 
 import com.jme3.network.HostedConnection;
 import com.jme3.network.serializing.Serializer;
+import com.jme3.network.serializing.serializers.FieldSerializer;
 import com.jme3.network.service.AbstractHostedConnectionService;
 import com.jme3.network.service.HostedServiceManager;
 import com.jme3.network.util.SessionDataDelegator;
@@ -88,7 +89,8 @@ public EtherealHost( ObjectStateProtocol objectProtocol, ZoneGrid grid, Vec3i cl
         this.grid = grid;
         this.clientZoneExtents = clientZoneExtents;
 
-        Serializer.registerClasses(ClientStateMessage.class, ObjectStateMessage.class);        
+        Serializer.registerClasses(ClientStateMessage.class, ObjectStateMessage.class); 
+        Serializer.registerClass(Vec3d.class, new FieldSerializer());        
     }
 
     public ZoneManager getZones() {
@@ -191,7 +193,7 @@ public void startHostingOnConnection( HostedConnection hc ) {
             return;
         }
 
-        nsl = new NetworkStateListener(this, hc, grid, 1);
+        nsl = new NetworkStateListener(this, hc, grid, clientZoneExtents);
         hc.setAttribute(NetworkStateListener.ATTRIBUTE_KEY, nsl);
 
         // Now add it to the state collector

src/main/java/com/simsilica/ethereal/NetworkStateListener.java

@@ -38,6 +38,7 @@
 
 import com.jme3.network.HostedConnection;
 import com.simsilica.mathd.Quatd;
+import com.simsilica.mathd.Vec3i;
 import com.simsilica.mathd.Vec3d;
 import com.simsilica.ethereal.net.ClientStateMessage;
 import com.simsilica.ethereal.net.SentState;
@@ -108,6 +109,10 @@ public class NetworkStateListener implements StateListener {
     public NetworkStateListener( EtherealHost host, HostedConnection conn, ZoneGrid grid, int zoneRadius ) {
         this(host, conn, new LocalZoneIndex(grid, zoneRadius), new IdIndex(10));
     }
+
+    public NetworkStateListener( EtherealHost host, HostedConnection conn, ZoneGrid grid, Vec3i zoneExtents ) {
+        this(host, conn, new LocalZoneIndex(grid, zoneExtents), new IdIndex(10));
+    }
 
     public NetworkStateListener( EtherealHost host, HostedConnection conn, LocalZoneIndex zoneIndex, IdIndex idIndex ) {
         this.host = host;
@@ -221,6 +226,10 @@ public void endFrameBlock() {
 
     @Override
     public void beginFrame( long time ) {
+ 
+        if( log.isTraceEnabled() ) {
+            log.trace(self + ":beginFrame(" + time + ") selfPosition:" + selfPosition);
+        }
 
         // If the zones changed last frame then clear the buffers
         if( zonesChanged ) {    
@@ -233,6 +242,10 @@ public void beginFrame( long time ) {
     @Override
     public void endFrame( long time ) {
 
+        if( log.isTraceEnabled() ) {
+            log.trace(self + ":endFrame(" + time + ") selfPosition:" + selfPosition);
+        }
+        
         // See if we've gotten any ACKs to add to our ACK header
         ClientStateMessage ackedMsg;
         while( (ackedMsg = acked.poll()) != null ) {
@@ -317,6 +330,10 @@ public void endFrame( long time ) {
     @Override
     public void stateChanged( StateBlock b ) {
 
+        if( log.isTraceEnabled() ) {
+            log.trace(self + ":stateChanged(" + b + ")");
+        }
+
         // StateBlocks are per zone but contain all of the object
         // objects for a particular time step in that zone.
 
@@ -346,6 +363,10 @@ public void stateChanged( StateBlock b ) {
                     }
                 }                 
             }                
+        } else {
+            if( log.isTraceEnabled() ) {
+                log.trace(self + ":No updates");
+            }
         }
         if( b.getRemovals() != null ) {
             for( Long e : b.getRemovals() )