[Java] Add some missing use of Serializable. Issue #223.

real-logic · Oct 2, 2020 · 9f4b17f · 9f4b17f
1 parent cb97cda
commit 9f4b17f
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 21 deletions.
diff --git a/agrona/src/main/java/org/agrona/collections/Int2IntHashMap.java b/agrona/src/main/java/org/agrona/collections/Int2IntHashMap.java
@@ -806,7 +806,7 @@ public void remove()
     /**
      * Iterator over keys which supports access to unboxed keys via {@link #nextValue()}.
      */
-    public final class KeyIterator extends AbstractIterator implements Iterator<Integer>
+    public final class KeyIterator extends AbstractIterator implements Iterator<Integer>, Serializable
     {
         public Integer next()
         {
@@ -828,7 +828,7 @@ public int nextValue()
     /**
      * Iterator over values which supports access to unboxed values.
      */
-    public final class ValueIterator extends AbstractIterator implements Iterator<Integer>
+    public final class ValueIterator extends AbstractIterator implements Iterator<Integer>, Serializable
     {
         public Integer next()
         {
@@ -852,7 +852,7 @@ public int nextValue()
      */
     public final class EntryIterator
         extends AbstractIterator
-        implements Iterator<Entry<Integer, Integer>>, Entry<Integer, Integer>
+        implements Iterator<Entry<Integer, Integer>>, Entry<Integer, Integer>, Serializable
     {
         public Integer getKey()
         {
@@ -1087,7 +1087,7 @@ public boolean contains(final int key)
     /**
      * Collection of values which supports optionally cached iterators to avoid allocation.
      */
-    public final class ValueCollection extends AbstractCollection<Integer>
+    public final class ValueCollection extends AbstractCollection<Integer> implements Serializable
     {
         private final ValueIterator valueIterator = shouldAvoidAllocation ? new ValueIterator() : null;
 

diff --git a/agrona/src/main/java/org/agrona/collections/Int2ObjectCache.java b/agrona/src/main/java/org/agrona/collections/Int2ObjectCache.java
@@ -628,7 +628,7 @@ public void clear()
     /**
      * Collection of values which supports cached iterator to avoid allocation.
      */
-    public final class ValueCollection extends AbstractCollection<V>
+    public final class ValueCollection extends AbstractCollection<V> implements Serializable
     {
         private final ValueIterator iterator = new ValueIterator();
 
@@ -658,7 +658,7 @@ public void clear()
     /**
      * Set of entries which supports cached iterator to avoid allocation.
      */
-    public final class EntrySet extends AbstractSet<Map.Entry<Integer, V>>
+    public final class EntrySet extends AbstractSet<Map.Entry<Integer, V>> implements Serializable
     {
         private final EntryIterator iterator = new EntryIterator();
 
@@ -751,7 +751,7 @@ void reset()
     /**
      * An iterator over values.
      */
-    public final class ValueIterator extends AbstractIterator<V>
+    public final class ValueIterator extends AbstractIterator<V> implements Serializable
     {
         @SuppressWarnings("unchecked")
         public V next()
@@ -764,7 +764,7 @@ public V next()
     /**
      * Iterator over keys which supports access to unboxed keys via {@link #nextInt()}.
      */
-    public final class KeyIterator extends AbstractIterator<Integer>
+    public final class KeyIterator extends AbstractIterator<Integer> implements Serializable
     {
         public Integer next()
         {
@@ -788,7 +788,7 @@ public int nextInt()
      */
     public final class EntryIterator
         extends AbstractIterator<Entry<Integer, V>>
-        implements Entry<Integer, V>
+        implements Entry<Integer, V>, Serializable
     {
         public Entry<Integer, V> next()
         {

diff --git a/agrona/src/main/java/org/agrona/collections/Int2ObjectHashMap.java b/agrona/src/main/java/org/agrona/collections/Int2ObjectHashMap.java
@@ -994,7 +994,7 @@ final void reset()
     /**
      * Iterator over values.
      */
-    public class ValueIterator extends AbstractIterator<V>
+    public class ValueIterator extends AbstractIterator<V> implements Serializable
     {
         public V next()
         {
@@ -1007,7 +1007,7 @@ public V next()
     /**
      * Iterator over keys which supports access to unboxed keys via {@link #nextInt()}.
      */
-    public class KeyIterator extends AbstractIterator<Integer>
+    public class KeyIterator extends AbstractIterator<Integer> implements Serializable
     {
         public Integer next()
         {
@@ -1032,7 +1032,7 @@ public int nextInt()
      */
     public class EntryIterator
         extends AbstractIterator<Entry<Integer, V>>
-        implements Entry<Integer, V>
+        implements Entry<Integer, V>, Serializable
     {
         public Entry<Integer, V> next()
         {
@@ -1091,7 +1091,7 @@ public V setValue(final V value)
         /**
          * An {@link java.util.Map.Entry} implementation.
          */
-        public final class MapEntry implements Entry<Integer, V>
+        public final class MapEntry implements Entry<Integer, V>, Serializable
         {
             private final int k;
             private final V v;

diff --git a/agrona/src/main/java/org/agrona/collections/Object2IntHashMap.java b/agrona/src/main/java/org/agrona/collections/Object2IntHashMap.java
@@ -950,7 +950,7 @@ final void reset()
     /**
      * Iterator over values providing unboxed access via {@link #nextInt()}.
      */
-    public final class ValueIterator extends AbstractIterator<Integer>
+    public final class ValueIterator extends AbstractIterator<Integer> implements Serializable
     {
         public Integer next()
         {
@@ -972,7 +972,7 @@ public int nextInt()
     /**
      * Iterator over keys.
      */
-    public final class KeyIterator extends AbstractIterator<K>
+    public final class KeyIterator extends AbstractIterator<K> implements Serializable
     {
         public K next()
         {
@@ -986,7 +986,7 @@ public K next()
      */
     public final class EntryIterator
         extends AbstractIterator<Entry<K, Integer>>
-        implements Entry<K, Integer>
+        implements Entry<K, Integer>, Serializable
     {
         public Entry<K, Integer> next()
         {
@@ -1053,7 +1053,7 @@ public int setValue(final int value)
         /**
          * An {@link java.util.Map.Entry} implementation.
          */
-        public final class MapEntry implements Entry<K, Integer>
+        public final class MapEntry implements Entry<K, Integer>, Serializable
         {
             private final K k;
             private final int v;

diff --git a/agrona/src/main/java/org/agrona/collections/Object2ObjectHashMap.java b/agrona/src/main/java/org/agrona/collections/Object2ObjectHashMap.java
@@ -632,7 +632,7 @@ public void remove()
     /**
      * An iterator over keys.
      */
-    public final class KeyIterator extends AbstractIterator implements Iterator<K>
+    public final class KeyIterator extends AbstractIterator implements Iterator<K>, Serializable
     {
         @SuppressWarnings("unchecked")
         public K next()
@@ -645,7 +645,7 @@ public K next()
     /**
      * An iterator over values.
      */
-    public final class ValueIterator extends AbstractIterator implements Iterator<V>
+    public final class ValueIterator extends AbstractIterator implements Iterator<V>, Serializable
     {
         public V next()
         {
@@ -659,7 +659,7 @@ public V next()
      */
     public final class EntryIterator
         extends AbstractIterator
-        implements Iterator<Entry<K, V>>, Entry<K, V>
+        implements Iterator<Entry<K, V>>, Entry<K, V>, Serializable
     {
         @SuppressWarnings("unchecked")
         public K getKey()
@@ -869,7 +869,7 @@ public void forEach(final Consumer<? super K> action)
     /**
      * A collection of values.
      */
-    public final class ValueCollection extends AbstractCollection<V>
+    public final class ValueCollection extends AbstractCollection<V> implements Serializable
     {
         private final ValueIterator valueIterator = shouldAvoidAllocation ? new ValueIterator() : null;