LGTM with comments
http://gwt-code-reviews.appspot.com/367803/diff/1/3 File /bikeshed/src/com/google/gwt/collections/Map.java (right): http://gwt-code-reviews.appspot.com/367803/diff/1/3#newcode35 /bikeshed/src/com/google/gwt/collections/Map.java:35: public abstract boolean containsKey(K key); Can K be null? Need to document what happens. http://gwt-code-reviews.appspot.com/367803/diff/1/3#newcode43 /bikeshed/src/com/google/gwt/collections/Map.java:43: public abstract V get(K key); Can K be null? Need to document what happens. http://gwt-code-reviews.appspot.com/367803/diff/1/4 File /bikeshed/src/com/google/gwt/collections/MutableMap.java (right): http://gwt-code-reviews.appspot.com/367803/diff/1/4#newcode32 /bikeshed/src/com/google/gwt/collections/MutableMap.java:32: public abstract boolean containsKey(K key); No need to redeclare the three methods from Map. http://gwt-code-reviews.appspot.com/367803/diff/1/4#newcode47 /bikeshed/src/com/google/gwt/collections/MutableMap.java:47: public abstract void put(K key, V value); Can K or V be null? Need to document what happens. http://gwt-code-reviews.appspot.com/367803/diff/1/4#newcode54 /bikeshed/src/com/google/gwt/collections/MutableMap.java:54: public abstract void remove(K key); Can K be null? Need to document what happens. http://gwt-code-reviews.appspot.com/367803/diff/1/5 File /bikeshed/src/com/google/gwt/collections/MutableStringMap.java (right): http://gwt-code-reviews.appspot.com/367803/diff/1/5#newcode21 /bikeshed/src/com/google/gwt/collections/MutableStringMap.java:21: * A Map whose keys are supported by Strings. I'm assuming this is just the dev mode implementation with a JS implementation to follow? http://gwt-code-reviews.appspot.com/367803/diff/1/5#newcode56 /bikeshed/src/com/google/gwt/collections/MutableStringMap.java:56: initEntries(); Why not just inline initEntries here since this is the only call site http://gwt-code-reviews.appspot.com/367803/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
