LinkedList.java (clone): Clear the copy list with clear(), not by setting its size...

	* java/util/LinkedList.java (clone): Clear the copy list with clear(),
	not by setting its size field.

From-SVN: r44057

diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index 017a99a8cf0768a550bc64ce3d120a4eb096a657..161649f124a47e9a7f8704fe015998bcf627f758 100644 (file)
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -1,3 +1,8 @@
+2001-07-17  Bryce McKinlay  <bryce@waitaki.otago.ac.nz>
+
+       * java/util/LinkedList.java (clone): Clear the copy list with clear(), 
+       not by setting its size field.
+
 2001-07-12  Tom Tromey  <tromey@redhat.com>
            David Brownell  <david-b@pacbell.net>
        

diff --git a/libjava/java/util/LinkedList.java b/libjava/java/util/LinkedList.java
index 092c8bd7e739031c6e867b8cc5af1892fee4c5d9..41ca0936cf2123f2edbbe16cae28a4e42834acb4 100644 (file)
--- a/libjava/java/util/LinkedList.java
+++ b/libjava/java/util/LinkedList.java
@@ -474,7 +474,7 @@ public class LinkedList extends AbstractSequentialList
     catch (CloneNotSupportedException ex)
       {
       }
-    copy.size = 0;
+    copy.clear();
     copy.addAll(this);
     return copy;
   }
@@ -534,9 +534,7 @@ public class LinkedList extends AbstractSequentialList
   }
   
   /** A ListIterator over the list. This class keeps track of its
-   * position in the list, the size of the list, and the two list
-   * entries it is between.  This enables it to be used identically
-   * for both the list itself and a sublist of the list.
+   * position in the list and the two list entries it is between.
    */
   class LinkedListItr implements ListIterator
   {
@@ -546,10 +544,6 @@ public class LinkedList extends AbstractSequentialList
     Entry lastReturned; // entry that will be affected by remove() or set().
     int position;       // index of `next'.
 
-    /**
-     * Create a new Iter starting at a given Entry within the list, at a given
-     * position, in a list of given size.
-     */
     LinkedListItr(int index)
     {
       if (index == size)