Per Dave Hoover:
- 1.5 compatibility: Removed @Override on some methods that had no concrete implementation in superclasses. - Added @Override to a few places where there does happen to be an implementation in our abstract superclass. I now think the 1.6 behavior of @Override is more sensible, it's less special-casey. git-svn-id: https://samskivert.googlecode.com/svn/trunk@2653 6335cc39-0255-0410-8fd6-9bcaacd3b74c
This commit is contained in:
@@ -22,7 +22,6 @@ package com.samskivert.util;
|
|||||||
|
|
||||||
import java.io.Serializable;
|
import java.io.Serializable;
|
||||||
|
|
||||||
import java.util.AbstractSet;
|
|
||||||
import java.util.Arrays;
|
import java.util.Arrays;
|
||||||
import java.util.Collection;
|
import java.util.Collection;
|
||||||
import java.util.Iterator;
|
import java.util.Iterator;
|
||||||
@@ -232,7 +231,7 @@ public class ArrayIntSet extends AbstractIntSet
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
public void remove () {
|
@Override public void remove () {
|
||||||
if (_pos == 0) {
|
if (_pos == 0) {
|
||||||
throw new IllegalStateException();
|
throw new IllegalStateException();
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -396,7 +396,7 @@ public class HashIntMap<V> extends AbstractMap<Integer,V>
|
|||||||
// damn Sun bastards made the 'keySet' variable with default access, so we can't share it
|
// damn Sun bastards made the 'keySet' variable with default access, so we can't share it
|
||||||
if (_keySet == null) {
|
if (_keySet == null) {
|
||||||
_keySet = new AbstractIntSet() {
|
_keySet = new AbstractIntSet() {
|
||||||
@Override public Interator interator () {
|
public Interator interator () {
|
||||||
return new AbstractInterator () {
|
return new AbstractInterator () {
|
||||||
public boolean hasNext () {
|
public boolean hasNext () {
|
||||||
return i.hasNext();
|
return i.hasNext();
|
||||||
@@ -404,7 +404,7 @@ public class HashIntMap<V> extends AbstractMap<Integer,V>
|
|||||||
public int nextInt () {
|
public int nextInt () {
|
||||||
return i.next().getIntKey();
|
return i.next().getIntKey();
|
||||||
}
|
}
|
||||||
public void remove () {
|
@Override public void remove () {
|
||||||
i.remove();
|
i.remove();
|
||||||
}
|
}
|
||||||
private Iterator<IntEntry<V>> i = intEntrySet().iterator();
|
private Iterator<IntEntry<V>> i = intEntrySet().iterator();
|
||||||
|
|||||||
@@ -313,7 +313,7 @@ public class IntIntMap
|
|||||||
public IntSet keySet ()
|
public IntSet keySet ()
|
||||||
{
|
{
|
||||||
return new AbstractIntSet() {
|
return new AbstractIntSet() {
|
||||||
@Override public Interator interator () {
|
public Interator interator () {
|
||||||
return IntIntMap.this.keys();
|
return IntIntMap.this.keys();
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -321,11 +321,11 @@ public class IntIntMap
|
|||||||
return IntIntMap.this.size();
|
return IntIntMap.this.size();
|
||||||
}
|
}
|
||||||
|
|
||||||
public boolean contains (int t) {
|
@Override public boolean contains (int t) {
|
||||||
return IntIntMap.this.containsKey(t);
|
return IntIntMap.this.containsKey(t);
|
||||||
}
|
}
|
||||||
|
|
||||||
public boolean remove (int value) {
|
@Override public boolean remove (int value) {
|
||||||
// we have to check for presence in the map separately because we have no "not in
|
// we have to check for presence in the map separately because we have no "not in
|
||||||
// the set" return value
|
// the set" return value
|
||||||
if (!IntIntMap.this.containsKey(value)) {
|
if (!IntIntMap.this.containsKey(value)) {
|
||||||
@@ -566,7 +566,7 @@ public class IntIntMap
|
|||||||
return _eiter.hasNext();
|
return _eiter.hasNext();
|
||||||
}
|
}
|
||||||
|
|
||||||
public void remove () {
|
@Override public void remove () {
|
||||||
_eiter.remove();
|
_eiter.remove();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user