Remove todos related to privatizing fields -> keep it as is for now.
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/collections/trunk@1493940 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
c5cc3a04e2
commit
a02dafa47d
|
@ -41,8 +41,6 @@ import org.apache.commons.collections4.keyvalue.AbstractMapEntryDecorator;
|
|||
*/
|
||||
public abstract class AbstractDualBidiMap<K, V> implements BidiMap<K, V> {
|
||||
|
||||
// TODO privatise fields?
|
||||
|
||||
/**
|
||||
* Normal delegate map.
|
||||
*/
|
||||
|
|
|
@ -54,8 +54,6 @@ public abstract class AbstractLinkedList<E> implements List<E> {
|
|||
* - respects {@link AbstractList#modCount}
|
||||
*/
|
||||
|
||||
// TODO Privatise fields?
|
||||
|
||||
/**
|
||||
* A {@link Node} which indicates the start and end of the list and does not
|
||||
* hold a value. The value of <code>next</code> is the first item in the
|
||||
|
|
|
@ -75,8 +75,6 @@ public class AbstractHashedMap<K, V> extends AbstractMap<K, V> implements Iterab
|
|||
/** An object for masking null */
|
||||
protected static final Object NULL = new Object();
|
||||
|
||||
// TODO Privatise fields?
|
||||
|
||||
/** Load factor, normally 0.75 */
|
||||
protected transient float loadFactor;
|
||||
/** The size of the map */
|
||||
|
|
|
@ -62,7 +62,7 @@ import org.apache.commons.collections4.iterators.EmptyOrderedMapIterator;
|
|||
public abstract class AbstractLinkedMap<K, V> extends AbstractHashedMap<K, V> implements OrderedMap<K, V> {
|
||||
|
||||
/** Header in the linked list */
|
||||
protected transient LinkEntry<K, V> header; // TODO Privatise?
|
||||
protected transient LinkEntry<K, V> header;
|
||||
|
||||
/**
|
||||
* Constructor only used in deserialization, do not use otherwise.
|
||||
|
@ -501,7 +501,7 @@ public abstract class AbstractLinkedMap<K, V> extends AbstractHashedMap<K, V> im
|
|||
*/
|
||||
protected static class LinkEntry<K, V> extends HashEntry<K, V> {
|
||||
/** The entry before this one in the order */
|
||||
protected LinkEntry<K, V> before; // TODO Privatise?
|
||||
protected LinkEntry<K, V> before;
|
||||
/** The entry after this one in the order */
|
||||
protected LinkEntry<K, V> after;
|
||||
|
||||
|
|
|
@ -31,8 +31,6 @@ import org.apache.commons.collections4.ResettableIterator;
|
|||
*/
|
||||
public class EntrySetToMapIteratorAdapter<K, V> implements MapIterator<K, V>, ResettableIterator<K> {
|
||||
|
||||
// TODO Privatise fields?
|
||||
|
||||
/** The adapted Map entry Set. */
|
||||
protected Set<Map.Entry<K, V>> entrySet;
|
||||
|
||||
|
|
Loading…
Reference in New Issue