marcwrobel commented on code in PR #325:
URL: 
https://github.com/apache/commons-collections/pull/325#discussion_r936015101


##########
src/site/xdoc/pick.xml:
##########
@@ -112,66 +112,66 @@ The <a 
href="https://commons.apache.org/collections/apidocs-COLLECTIONS_3_1/org/
 <h3>Map interface</h3>
 
 <p>
-The <a href="http://java.sun.com/j2se/1.4/docs/api/java/util/Map.html";>Map</a> 
interface and implementations in both the JDK and Commons Collections allow you 
to lookup data from a key to a value. This is one of the most powerful 
interfaces in the JDK, however it is very difficult to implement. These are the 
available implementations:
+The <a 
href="https://docs.oracle.com/javase/8/docs/api/java/util/Map.html";>Map</a> 
interface and implementations in both the JDK and Commons Collections allow you 
to lookup data from a key to a value. This is one of the most powerful 
interfaces in the JDK, however it is very difficult to implement. These are the 
available implementations:
 <table>
 <tr>
-<td><a 
href="http://java.sun.com/j2se/1.4/docs/api/java/util/HashMap.html";>HashMap</a> 
(JDK)</td>
+<td><a 
href="https://docs.oracle.com/javase/8/docs/api/java/util/HashMap.html";>HashMap</a>
 (JDK)</td>
 <td>This map is the most commonly used and fastest implementation. It is 
suitable for most situations and can store any object. Objects are stored in 
any order, and the order may change over time.</td>
 </tr>
 <tr>
-<td><a 
href="https://commons.apache.org/collections/apidocs-COLLECTIONS_3_1/org/apache/commons/collections/map/HashedMap.html";>HashedMap</a></td>
-<td>The Commons Collections hash map implementation which is very similar in 
design to HashMap, but also supports easy iteration via a <a 
href="https://commons.apache.org/collections/apidocs-COLLECTIONS_3_1/org/apache/commons/collections/MapIterator.html";>MapIterator</a>.
 This implementation is also designed to be subclassed. Use this map if you 
want to use the extra iterator, or to subclass. Otherwise, there is no 
advantage (or disadvantage) compared to the JDK HashMap.</td>
+<td><a 
href="https://javadoc.io/doc/commons-collections/commons-collections/3.1/org/apache/commons/collections/map/HashedMap.html";>HashedMap</a></td>
+<td>The Commons Collections hash map implementation which is very similar in 
design to HashMap, but also supports easy iteration via a <a 
href="https://javadoc.io/doc/commons-collections/commons-collections/3.1/org/apache/commons/collections/MapIterator.html";>MapIterator</a>.
 This implementation is also designed to be subclassed. Use this map if you 
want to use the extra iterator, or to subclass. Otherwise, there is no 
advantage (or disadvantage) compared to the JDK HashMap.</td>
 </tr>
 <tr>
-<td><a 
href="https://commons.apache.org/collections/apidocs-COLLECTIONS_3_1/org/apache/commons/collections/map/IdentityMap.html";>IdentityMap</a></td>
+<td><a 
href="https://javadoc.io/doc/commons-collections/commons-collections/3.1/org/apache/commons/collections/map/IdentityMap.html";>IdentityMap</a></td>
 <td>This map operates exactly as per HashedMap but compares keys and values 
using == not .equals().</td>
 </tr>
 <tr>
-<td><a 
href="http://java.sun.com/j2se/1.4/docs/api/java/util/IdentityHashMap.html";>IdentityHashMap</a>
 (JDK)</td>
+<td><a 
href="https://docs.oracle.com/javase/8/docs/api/java/util/IdentityHashMap.html";>IdentityHashMap</a>
 (JDK)</td>
 <td>This map operates exactly as per HashMap but compares keys and values 
using == not .equals().</td>
 </tr>
 <tr>
-<td><a 
href="http://java.sun.com/j2se/1.4/docs/api/java/util/TreeMap.html";>TreeMap</a> 
(JDK)</td>
-<td>This map ensures that the keys are always sorted, and iteration order is 
consistent. All keys must implement <a 
href="http://java.sun.com/j2se/1.4/docs/api/java/lang/Comparable.html";>Comparable</a>,
 unless a comparator is supplied. This map should be used if you need the map 
to be sorted.</td>
+<td><a 
href="https://docs.oracle.com/javase/8/docs/api/java/util/TreeMap.html";>TreeMap</a>
 (JDK)</td>
+<td>This map ensures that the keys are always sorted, and iteration order is 
consistent. All keys must implement <a 
href="https://docs.oracle.com/javase/8/docs/api/java/lang/Comparable.html";>Comparable</a>,
 unless a comparator is supplied. This map should be used if you need the map 
to be sorted.</td>
 </tr>
 <tr>
-<td><a 
href="https://commons.apache.org/collections/apidocs-COLLECTIONS_3_1/org/apache/commons/collections/map/LinkedMap.html";>LinkedMap</a></td>
+<td><a 
href="https://javadoc.io/doc/commons-collections/commons-collections/3.1/org/apache/commons/collections/map/LinkedMap.html";>LinkedMap</a></td>
 <td>This map maintains the order that each object is added to the map. When 
the map is viewed via an iterator, the insertion order will be seen. This map 
should be used if you need to retain the insertion order.</td>
 </tr>
 <tr>
-<td><a 
href="http://java.sun.com/j2se/1.4/docs/api/java/util/LinkedHashMap.html";>LinkedHashMap</a>
 (JDK)</td>
+<td><a 
href="https://docs.oracle.com/javase/8/docs/api/java/util/LinkedHashMap.html";>LinkedHashMap</a>
 (JDK)</td>
 <td>This map, available from JDK 1.4, maintains the order that each object is 
added to the map. When the map is viewed via an iterator, the insertion order 
will be seen. Commons collections' LinkedMap provides the same functionality 
for JDK1.2 onwards and provides a MapIterator.</td>
 </tr>
 <tr>
-<td><a 
href="https://commons.apache.org/collections/apidocs-COLLECTIONS_3_1/org/apache/commons/collections/map/ListOrderedMap.html";>ListOrderedMap</a></td>
+<td><a 
href="https://javadoc.io/doc/commons-collections/commons-collections/3.1/org/apache/commons/collections/map/ListOrderedMap.html";>ListOrderedMap</a></td>
 <td>This map decorates another map to maintains the order that each object is 
added to the map. The order is maintained using a list. This map should be used 
if you need to retain the insertion order but you also need the special 
features of another map.</td>
 </tr>
 <tr>
-<td><a 
href="https://commons.apache.org/collections/apidocs-COLLECTIONS_3_1/org/apache/commons/collections/map/LRUMap.html";>LRUMap</a></td>
+<td><a 
href="https://javadoc.io/doc/commons-collections/commons-collections/3.1/org/apache/commons/collections/map/LRUMap.html";>LRUMap</a></td>
 <td>This map places a maximum size limit on the map and removes the least 
recently used (LRU) key-value when it is full. This map should be used for 
caches where you want to limit the maximum memory size used, and are happy with 
the least recently used algorithm for removals.</td>
 </tr>
 <tr>
-<td><a 
href="https://commons.apache.org/collections/apidocs-COLLECTIONS_3_1/org/apache/commons/collections/map/ReferenceMap.html";>ReferenceMap</a></td>
+<td><a 
href="https://javadoc.io/doc/commons-collections/commons-collections/3.1/org/apache/commons/collections/map/ReferenceMap.html";>ReferenceMap</a></td>
 <td>This map enables keys and values to be garbage collected whilst still held 
in the map. This can be useful for building memory sensitive caches. This map 
should be used for caches where you want to allow garbage collection from the 
map.</td>
 </tr>
 <tr>
-<td><a 
href="http://java.sun.com/j2se/1.4/docs/api/java/util/WeakHashMap.html";>WeakHashMap</a>
 (JDK)</td>
+<td><a 
href="https://docs.oracle.com/javase/8/docs/api/java/util/WeakHashMap.html";>WeakHashMap</a>
 (JDK)</td>
 <td>This map stores each key-value pair with a key that can be garbage 
collected. This can be useful for building memory sensitive caches. 
ReferenceMap provides the same functionality but with much more 
flexibility.</td>
 </tr>
 <tr>
-<td><a 
href="https://commons.apache.org/collections/apidocs-COLLECTIONS_3_1/org/apache/commons/collections/map/ReferenceIdentityMap.html";>ReferenceIdentityMap</a></td>
+<td><a 
href="https://javadoc.io/doc/commons-collections/commons-collections/3.1/org/apache/commons/collections/map/ReferenceIdentityMap.html";>ReferenceIdentityMap</a></td>
 <td>This map operates exactly as per ReferenceMap but compares keys and values 
using == not .equals().</td>
 </tr>
 <tr>
-<td><a 
href="https://commons.apache.org/collections/apidocs-COLLECTIONS_3_1/org/apache/commons/collections/map/SingletonMap.html";>SingletonMap</a></td>
+<td><a 
href="https://javadoc.io/doc/commons-collections/commons-collections/3.1/org/apache/commons/collections/map/SingletonMap.html";>SingletonMap</a></td>
 <td>This map is restricted to storing one key-value pair. It may contain no 
more than and no less than one pair. It provides a MapIterator. This map should 
be used if you want to return one key-value pair and must use the Map 
interface.</td>
 </tr>
 <tr>
-<td><a 
href="https://commons.apache.org/collections/apidocs-COLLECTIONS_3_1/org/apache/commons/collections/map/Flat3Map.html";>Flat3Map</a></td>
+<td><a 
href="https://javadoc.io/doc/commons-collections/commons-collections/3.1/org/apache/commons/collections/map/Flat3Map.html";>Flat3Map</a></td>
 <td>This map is optimised to store one, two or three key-value pairs and 
outperforms HashMap at these sizes. For size four and above performance is 
about 5% slower than HashMap. This map also has good garbage collection 
characteristics when below size four. It provides a MapIterator. This map 
should be used if are 99% sure that the map will be size three or less.</td>
 </tr>
 <tr>
-<td><a 
href="https://commons.apache.org/collections/apidocs-COLLECTIONS_3_1/org/apache/commons/collections/map/StaticBucketMap.html";>StaticBucketMap</a></td>
+<td><a 
href="https://javadoc.io/doc/commons-collections/commons-collections/3.1/org/apache/commons/collections/map/StaticBucketMap.html";>StaticBucketMap</a></td>

Review Comment:
   @kinow, unfortunately the javadoc for commons-collection 3.1 is not 
available on https://commons.apache.org/proper/commons-collections/javadocs/. 
Should I use links to the 3.2.2 ?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to