These inner classes should be static. In the case of IntIntMap this was a big

fat waste of four bytes per map entry. Whoops.


git-svn-id: https://samskivert.googlecode.com/svn/trunk@2732 6335cc39-0255-0410-8fd6-9bcaacd3b74c

src/java/com/samskivert/net/cddb/CDDB.java

@@ -460,7 +460,7 @@ public class CDDB
     /**
      * A simple class to encapsulate the response from the CDDB server.
      */
-    protected class Response
+    protected static class Response
     {
         public int code;
         public String message;

src/java/com/samskivert/swing/TableSorter.java

@@ -336,7 +336,7 @@ public class TableSorter extends AbstractTableModel {
 
     // Helper classes
 
-    protected class Row {
+    protected static class Row {
         public final int modelIndex;
         public Row (int index) {
             this.modelIndex = index;

src/java/com/samskivert/util/IntIntMap.java

@@ -443,7 +443,7 @@ public class IntIntMap
         }
     }
 
-    class Record implements IntIntEntry
+    protected static class Record implements IntIntEntry
     {
 	public Record next;
 	public int key;
@@ -550,7 +550,7 @@ public class IntIntMap
         private int _modCount;
     }
 
-    protected class KeyValueInterator extends AbstractInterator
+    protected static class KeyValueInterator extends AbstractInterator
     {
         public KeyValueInterator (boolean keys, IntEntryIterator eiter) {
             _keys = keys;