Hi,

It seems like linkedList.clear() can be easily fixed to O(1) instead of O(n).

Meaning:


      308     public void clear() {

      309         Entry<E> e = header.next;

      310         while (e != header) {

      311             Entry<E> next = e.next;

      312             e.next = e.previous = null;

      313             e.element = null;

      314             e = next;

      315         }

      316         header.next = header.previous = header;

      317         size = 0;

      318         modCount++;

      319     }


A better implementations would be:


      308     public void clear() {

      309         header.next = header.previous = header;

      310         size = 0;

      311         modCount++;

      312     }


Guy

Reply via email to