Device is immutable. The new device needs to be returned when aged network address or AP is removed

src/main/java/net/floodlightcontroller/devicemanager/internal/DeviceManagerImpl.java

@@ -1763,11 +1763,15 @@ public class DeviceManagerImpl implements IDeviceManager, IOFMessageListener,
         deviceUpdateTask.reschedule(5, TimeUnit.SECONDS);          
     }
 
-    // ********************
-    // Device aging methods
-    // ********************    
+    /**
+     * Remove aged network address from device
+     *    
+     * @param device
+     * @param currentDate
+     * @return the new device object since the device is immutable
+     */
 
-    private void removeAgedNetworkAddresses(Device device, Date currentDate) {
+    private Device removeAgedNetworkAddresses(Device device, Date currentDate) {
         Collection<DeviceNetworkAddress> addresses = 
                                                 device.getNetworkAddresses();
 
@@ -1785,9 +1789,18 @@ public class DeviceManagerImpl implements IDeviceManager, IOFMessageListener,
                 removeNetworkAddressFromStorage(device, address);
             }
         }
+        
+        return devMgrMaps.getDeviceByDataLayerAddr(device.getDataLayerAddressAsLong());
     }
 
-    private void removeAgedAttachmentPoints(Device device, Date currentDate) {
+    /**
+     * Remove aged device attachment point
+     * 
+     * @param device
+     * @param currentDate
+     * @return the new device object since the device is immutable
+     */
+    private Device removeAgedAttachmentPoints(Device device, Date currentDate) {
         Collection<DeviceAttachmentPoint> aps = device.getAttachmentPoints();
 
         for (DeviceAttachmentPoint ap : aps) {
@@ -1804,6 +1817,8 @@ public class DeviceManagerImpl implements IDeviceManager, IOFMessageListener,
                 removeAttachmentPointFromStorage(device, ap);
             }
         }
+        
+        return devMgrMaps.getDeviceByDataLayerAddr(device.getDataLayerAddressAsLong());
     }
 
     /**
@@ -1815,8 +1830,8 @@ public class DeviceManagerImpl implements IDeviceManager, IOFMessageListener,
 
         Collection<Device> deviceColl = devMgrMaps.getDevices();
         for (Device device: deviceColl) {
-            removeAgedNetworkAddresses(device, currentDate);
-            removeAgedAttachmentPoints(device, currentDate);
+            device = removeAgedNetworkAddresses(device, currentDate);
+            device = removeAgedAttachmentPoints(device, currentDate);
 
             if ((device.getAttachmentPoints().size() == 0) &&
                 (device.getNetworkAddresses().size() == 0) &&