<p>This method creates a synchronized collection and store the reference to it
in a local variable. As local variables are by definition threadsafe, it seems
questionable that this collection needs to be synchronized.</p>
<p>
<table>
  <tr><th>If you are using</th><th>consider using</th></tr>
  <tr><td>java.util.Vector</td><td>java.util.ArrayList</td></tr>
  <tr><td>java.util.Hashtable</td><td>java.util.HashMap</td></tr>
  <tr><td>java.lang.StringBuffer</td><td>java.lang.StringBuilder</td></tr>
</table>
</p>