Re: Vector (was Re: Change character in string)

Andreas Leitgeb wrote:


Here is the source of the underlying 'SynchronizedList' nested class's
'equals()' method:

   public boolean equals(Object o) {
     synchronized(mutex) {return list.equals(o);}
   }

That's good, because if 'equals()' weren't synchronized it would be broken.

I am grateful to Bent also, because I never knew about this flaw with
'Vector'. It is good to have more than just an aesthetic argument for my choice.

Now I have quite the mental exercise ahead of me to figure out why
'Vector#equals()' deadlocks and 'SynchronizedList#equals()' doesn't.

Here's the source from 'Vector':

public class Vector<E>
   extends AbstractList<E>
   implements List<E>, RandomAccess, Cloneable, java.io.Serializable
{
  ...
   public synchronized boolean equals(Object o) {
     return super.equals(o);
   }
  ...
}

How interesting that Sun, who usually espouses
"opening-brace-on-the-same-line", gives it a separate line at the class level
here.

--
Lew