AutoRefactor: use Collection.addAll()

Jean-Noel Rouvignac

13.40.2015 98e8aab354a385055392de7154758c1890a3265a

AutoRefactor: use Collection.addAll()

 opendj-server-legacy/src/main/java/org/opends/admin/ads/ReplicaDescriptor.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2007-2009 Sun Microsystems, Inc.
 *      Portions copyright 2013 ForgeRock, AS.
 *      Portions Copyright 2013-2015 ForgeRock AS.
 */

package org.opends.admin.ads;
@@ -135,9 +135,7 @@
   */
  public Set<String> getReplicationServers()
  {
    HashSet<String> copy = new HashSet<String>();
    copy.addAll(replicationServers);
    return copy;
    return new HashSet<String>(replicationServers);
  }

  /**

 opendj-server-legacy/src/main/java/org/opends/admin/ads/ServerDescriptor.java

@@ -133,9 +133,7 @@
   */
  public Set<ReplicaDescriptor> getReplicas()
  {
    Set<ReplicaDescriptor> copy = new HashSet<ReplicaDescriptor>();
    copy.addAll(replicas);
    return copy;
    return new HashSet<ReplicaDescriptor>(replicas);
  }

  /**

 opendj-server-legacy/src/main/java/org/opends/admin/ads/SuffixDescriptor.java

@@ -22,6 +22,7 @@
 *
 *
 *      Copyright 2008 Sun Microsystems, Inc.
 *      Portions Copyright 2015 ForgeRock AS
 */

package org.opends.admin.ads;
@@ -64,9 +65,7 @@
   */
  public Set<ReplicaDescriptor> getReplicas()
  {
    Set<ReplicaDescriptor> copy = new HashSet<ReplicaDescriptor>();
    copy.addAll(replicas);
    return copy;
    return new HashSet<ReplicaDescriptor>(replicas);
  }

  /**

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/datamodel/TaskTableModel.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2009 Sun Microsystems, Inc.
 *      Portions Copyright 2014 ForgeRock AS
 *      Portions Copyright 2014-2015 ForgeRock AS
 */
package org.opends.guitools.controlpanel.datamodel;

@@ -139,10 +139,7 @@
    TreeSet<TaskEntry> sortedSet = new TreeSet<TaskEntry>(this);
    sortedSet.addAll(data);
    dataSourceArray.clear();
    for (TaskEntry task : sortedSet)
    {
      dataSourceArray.add(task);
    }
    dataSourceArray.addAll(sortedSet);
  }

  /**

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/task/CancelTaskTask.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2009 Sun Microsystems, Inc.
 *      Portions Copyright 2014 ForgeRock AS
 *      Portions Copyright 2014-2015 ForgeRock AS
 */

package org.opends.guitools.controlpanel.task;
@@ -196,8 +196,7 @@
            {
              getProgressDialog().appendProgressHtml("<br><br>");
            }
            ArrayList<String> args = new ArrayList<String>();
            args.addAll(getObfuscatedCommandLineArguments(arguments));
            ArrayList<String> args = new ArrayList<String>(getObfuscatedCommandLineArguments(arguments));
            printEquivalentCommandLine(getCommandLinePath("manage-tasks"),
                    args, INFO_CTRL_PANEL_EQUIVALENT_CMD_TO_CANCEL_TASK.get(
                        task.getId()));

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/task/DeleteBaseDNAndBackendTask.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2008-2009 Sun Microsystems, Inc.
 *      Portions Copyright 2014 ForgeRock AS
 *      Portions Copyright 2014-2015 ForgeRock AS
 */

package org.opends.guitools.controlpanel.task;
@@ -449,8 +449,7 @@
    {
      oldBaseDNs.add(baseDN.getDn());
    }
    LinkedList<DN> newBaseDNs = new LinkedList<DN>();
    newBaseDNs.addAll(oldBaseDNs);
    LinkedList<DN> newBaseDNs = new LinkedList<DN>(oldBaseDNs);
    ArrayList<DN> dnsToRemove = new ArrayList<DN>();
    for (BaseDNDescriptor baseDN : baseDNs)
    {
@@ -490,8 +489,7 @@
      (LocalDBBackendCfgClient)root.getBackend(
          baseDNs.iterator().next().getBackend().getBackendID());
    SortedSet<DN> oldBaseDNs = backend.getBaseDN();
    SortedSet<DN> newBaseDNs = new TreeSet<DN>();
    newBaseDNs.addAll(oldBaseDNs);
    SortedSet<DN> newBaseDNs = new TreeSet<DN>(oldBaseDNs);
    ArrayList<DN> dnsToRemove = new ArrayList<DN>();
    for (BaseDNDescriptor baseDN : baseDNs)
    {

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/task/DeleteEntryTask.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2008-2010 Sun Microsystems, Inc.
 *      Portions Copyright 2013-2014 ForgeRock AS
 *      Portions Copyright 2013-2015 ForgeRock AS
 */

package org.opends.guitools.controlpanel.task;
@@ -518,8 +518,7 @@
   */
  private void printEquivalentCommandToDelete(DN dn, boolean usingControl)
  {
    ArrayList<String> args = new ArrayList<String>();
    args.addAll(getObfuscatedCommandLineArguments(
    ArrayList<String> args = new ArrayList<String>(getObfuscatedCommandLineArguments(
        getConnectionCommandLineArguments(useAdminCtx, true)));
    args.add(getNoPropertiesFileArgument());
    if (usingControl)

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/task/NewEntryTask.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2008-2009 Sun Microsystems, Inc.
 *      Portions Copyright 2014 ForgeRock AS
 *      Portions Copyright 2014-2015 ForgeRock AS
 */

package org.opends.guitools.controlpanel.task;
@@ -311,8 +311,7 @@
   */
  private void printEquivalentCommand()
  {
    ArrayList<String> args = new ArrayList<String>();
    args.addAll(getObfuscatedCommandLineArguments(
    ArrayList<String> args = new ArrayList<String>(getObfuscatedCommandLineArguments(
        getConnectionCommandLineArguments(useAdminCtx, true)));
    args.add(getNoPropertiesFileArgument());
    args.add("--defaultAdd");

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/task/Task.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2008-2009 Sun Microsystems, Inc.
 *      Portions Copyright 2011-2014 ForgeRock AS
 *      Portions Copyright 2011-2015 ForgeRock AS
 */

package org.opends.guitools.controlpanel.task;
@@ -939,8 +939,7 @@
  protected void printEquivalentCommandToModify(String dn,
      Collection<ModificationItem> mods, boolean useAdminCtx)
  {
    ArrayList<String> args = new ArrayList<String>();
    args.addAll(getObfuscatedCommandLineArguments(
    ArrayList<String> args = new ArrayList<String>(getObfuscatedCommandLineArguments(
        getConnectionCommandLineArguments(useAdminCtx, true)));
    args.add(getNoPropertiesFileArgument());
    String equiv = getEquivalentCommandLine(getCommandLinePath("ldapmodify"),
@@ -1081,8 +1080,7 @@
  protected void printEquivalentRenameCommand(DN oldDN, DN newDN,
      boolean useAdminCtx)
  {
    ArrayList<String> args = new ArrayList<String>();
    args.addAll(getObfuscatedCommandLineArguments(
    ArrayList<String> args = new ArrayList<String>(getObfuscatedCommandLineArguments(
        getConnectionCommandLineArguments(useAdminCtx, true)));
    args.add(getNoPropertiesFileArgument());
    String equiv = getEquivalentCommandLine(getCommandLinePath("ldapmodify"),

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/ui/CustomAttributePanel.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2008-2010 Sun Microsystems, Inc.
 *      Portions Copyright 2011-2014 ForgeRock AS
 *      Portions Copyright 2011-2015 ForgeRock AS
 */
package org.opends.guitools.controlpanel.ui;

@@ -789,8 +789,7 @@
      for (int i=0; i<combos.length; i++)
      {
        DefaultComboBoxModel model = (DefaultComboBoxModel)combos[i].getModel();
        ArrayList<Object> el = new ArrayList<Object>();
        el.addAll(ruleNames.get(i));
        ArrayList<Object> el = new ArrayList<Object>(ruleNames.get(i));
        if (model.getSize() == 0)
        {
          el.add(0, NO_MATCHING_RULE);

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/ui/CustomObjectClassPanel.java

@@ -22,9 +22,8 @@
 *
 *
 *      Copyright 2008-2010 Sun Microsystems, Inc.
 *      Portions Copyright 2014 ForgeRock AS
 *      Portions Copyright 2014-2015 ForgeRock AS
 */

package org.opends.guitools.controlpanel.ui;

import static org.opends.messages.AdminToolMessages.*;
@@ -1005,23 +1004,24 @@

  private Set<AttributeType> getRequiredAttributes()
  {
    HashSet<AttributeType> attrs = new HashSet<AttributeType>();
    attrs.addAll(attributes.getSelectedListModel1().getData());
    attrs.removeAll(inheritedRequiredAttributes);
    return attrs;
    return intersect(attributes.getSelectedListModel1().getData(), inheritedRequiredAttributes);
  }

  private Set<AttributeType> getOptionalAttributes()
  {
    HashSet<AttributeType> attrs = new HashSet<AttributeType>();
    attrs.addAll(attributes.getSelectedListModel2().getData());
    attrs.removeAll(inheritedOptionalAttributes);
    return intersect(attributes.getSelectedListModel2().getData(), inheritedOptionalAttributes);
  }

  private Set<AttributeType> intersect(Set<AttributeType> set1, Set<AttributeType> set2)
  {
    HashSet<AttributeType> attrs = new HashSet<AttributeType>(set1);
    attrs.removeAll(set2);
    return attrs;
  }

  private ObjectClass getNewObjectClass()
  {
    ObjectClass newObjectClass = new ObjectClass("",
    return new ObjectClass("",
        getObjectClassName(),
        getAllNames(),
        getOID(),
@@ -1032,8 +1032,6 @@
        getObjectClassType(),
        obsolete.isSelected(),
        getExtraProperties());

    return newObjectClass;
  }

  private void updateAttributes()
@@ -1132,14 +1130,8 @@
    inheritedRequiredAttributes.clear();
    for (ObjectClass p : getObjectClassSuperiors())
    {
      for (AttributeType attr : p.getRequiredAttributeChain())
      {
        inheritedRequiredAttributes.add(attr);
      }
      for (AttributeType attr : p.getOptionalAttributeChain())
      {
        inheritedOptionalAttributes.add(attr);
      }
      inheritedRequiredAttributes.addAll(p.getRequiredAttributeChain());
      inheritedOptionalAttributes.addAll(p.getOptionalAttributeChain());
    }
    for (AttributeType attr : inheritedRequiredAttributes)
    {

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/ui/MonitoringAttributesViewPanel.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2009-2010 Sun Microsystems, Inc.
 *      Portions Copyright 2014 ForgeRock AS
 *      Portions Copyright 2014-2015 ForgeRock AS
 */
package org.opends.guitools.controlpanel.ui;

@@ -127,12 +127,9 @@
  * @param attributes the attributes that will be proposed to the user.
  *
  */
 protected MonitoringAttributesViewPanel(
     LinkedHashSet<T> attributes)
 protected MonitoringAttributesViewPanel(LinkedHashSet<T> attributes)
 {
   super();
   monitoringAttributes = new LinkedHashSet<T>();
   monitoringAttributes.addAll(attributes);
   monitoringAttributes = new LinkedHashSet<T>(attributes);
   createLayout();
 }


 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/ui/NewAttributePanel.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2008-2010 Sun Microsystems, Inc.
 *      Portions Copyright 2014 ForgeRock AS
 *      Portions Copyright 2014-2015 ForgeRock AS
 */
package org.opends.guitools.controlpanel.ui;

@@ -297,8 +297,7 @@
      for (int i=0; i<combos.length; i++)
      {
        DefaultComboBoxModel model = (DefaultComboBoxModel)combos[i].getModel();
        ArrayList<Object> el = new ArrayList<Object>();
        el.addAll(ruleNames.get(i));
        ArrayList<Object> el = new ArrayList<Object>(ruleNames.get(i));
        if (model.getSize() == 0)
        {
          el.add(0, NO_MATCHING_RULE);

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/ui/NewBaseDNPanel.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2008-2009 Sun Microsystems, Inc.
 *      Portions Copyright 2011-2014 ForgeRock AS
 *      Portions Copyright 2011-2015 ForgeRock AS
 */

package org.opends.guitools.controlpanel.ui;
@@ -378,8 +378,7 @@
        sortedBackends.add(backend.getBackendID());
      }
    }
    ArrayList<Object> newElements = new ArrayList<Object>();
    newElements.addAll(sortedBackends);
    ArrayList<Object> newElements = new ArrayList<Object>(sortedBackends);
    if (sortedBackends.size() > 0)
    {
      newElements.add(COMBO_SEPARATOR);

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/ui/NewObjectClassPanel.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2008-2010 Sun Microsystems, Inc.
 *      Portions Copyright 2014 ForgeRock AS
 *      Portions Copyright 2014-2015 ForgeRock AS
 */

package org.opends.guitools.controlpanel.ui;
@@ -501,14 +501,8 @@
        inheritedRequiredAttributes.clear();
        for (ObjectClass oc : superiors.getSelectedSuperiors())
        {
          for (AttributeType attr : oc.getRequiredAttributeChain())
          {
            inheritedRequiredAttributes.add(attr);
          }
          for (AttributeType attr : oc.getOptionalAttributeChain())
          {
            inheritedOptionalAttributes.add(attr);
          }
          inheritedRequiredAttributes.addAll(oc.getRequiredAttributeChain());
          inheritedOptionalAttributes.addAll(oc.getOptionalAttributeChain());
        }
        for (AttributeType attr : inheritedRequiredAttributes)
        {
@@ -752,24 +746,24 @@

  private Set<AttributeType> getRequiredAttributes()
  {
    HashSet<AttributeType> attrs = new HashSet<AttributeType>();
    attrs.addAll(attributes.getSelectedListModel1().getData());
    attrs.removeAll(inheritedRequiredAttributes);
    return attrs;
    return intersect(attributes.getSelectedListModel1().getData(), inheritedRequiredAttributes);
  }

  private Set<AttributeType> getOptionalAttributes()
  {
    HashSet<AttributeType> attrs = new HashSet<AttributeType>();
    attrs.addAll(attributes.getSelectedListModel2().getData());
    attrs.removeAll(inheritedOptionalAttributes);
    return intersect(attributes.getSelectedListModel2().getData(), inheritedOptionalAttributes);
  }

  private Set<AttributeType> intersect(Set<AttributeType> set1, Set<AttributeType> set2)
  {
    HashSet<AttributeType> attrs = new HashSet<AttributeType>(set1);
    attrs.removeAll(set2);
    return attrs;
  }

  /**
   * A renderer for the attribute lists.  The renderer basically marks the
   * inherited attributes with an asterisk.
   *
   */
  private class AttributeTypeCellRenderer implements ListCellRenderer
  {

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/ui/ObjectClassEditorPanel.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2008 Sun Microsystems, Inc.
 *      Portions Copyright 2014 ForgeRock AS
 *      Portions Copyright 2014-2015 ForgeRock AS
 */

package org.opends.guitools.controlpanel.ui;
@@ -158,8 +158,7 @@
  public void okClicked()
  {
    String struct = (String)  structural.getSelectedItem();
    TreeSet<String> aux = new TreeSet<String>();
    aux.addAll(auxiliary.getSelectedListModel().getData());
    TreeSet<String> aux = new TreeSet<String>(auxiliary.getSelectedListModel().getData());
    aux.add("top");
    ObjectClassValue newValue = new ObjectClassValue(struct, aux);
    valueChanged = !newValue.equals(value);

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/ui/SimplifiedViewEntryPanel.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2008-2010 Sun Microsystems, Inc.
 *      Portions Copyright 2013-2014 ForgeRock AS.
 *      Portions Copyright 2013-2015 ForgeRock AS.
 */
package org.opends.guitools.controlpanel.ui;

@@ -708,9 +708,7 @@

//  Get all attributes that the entry can have
    Set<String> attributes = new LinkedHashSet<String>();
    ArrayList<String> entryAttrs = new ArrayList<String>();
    entryAttrs.addAll(sr.getAttributeNames());

    ArrayList<String> entryAttrs = new ArrayList<String>(sr.getAttributeNames());
    ArrayList<String> attrsWithNoOptions = new ArrayList<String>();
    for (String attr : entryAttrs)
    {
@@ -823,11 +821,7 @@
          attrsWithNoOptions, false);


      for (String attr : entryAttrs)
      {
        attributes.add(attr);
      }

      attributes.addAll(entryAttrs);
      attributes.add("aci");

      // In read-only mode display only the attributes with values
@@ -1574,10 +1568,7 @@
          Object value = comp.getValue();
          if (value instanceof Collection<?>)
          {
            for (Object o : (Collection<?>)value)
            {
              values.add(o);
            }
            values.addAll((Collection<?>) value);
          }
          else
          {

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/ui/TableViewEntryPanel.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2008-2010 Sun Microsystems, Inc.
 *      Portions Copyright 2014 ForgeRock AS
 *      Portions Copyright 2014-2015 ForgeRock AS
 */
package org.opends.guitools.controlpanel.ui;

@@ -806,10 +806,7 @@
          {
            if (valuePair.value instanceof Collection<?>)
            {
              for (Object o : (Collection<?>)valuePair.value)
              {
                values.add(o);
              }
              values.addAll((Collection<?>) valuePair.value);
            }
            else
            {

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/ui/components/ObjectClassCellPanel.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2008 Sun Microsystems, Inc.
 *      Portions Copyright 2014 ForgeRock AS
 *      Portions Copyright 2014-2015 ForgeRock AS
 */

package org.opends.guitools.controlpanel.ui.components;
@@ -192,8 +192,7 @@
    LocalizableMessageBuilder sb = new LocalizableMessageBuilder();
    if (value != null)
    {
      Set<String> aux = new TreeSet<String>();
      aux.addAll(value.getAuxiliary());
      Set<String> aux = new TreeSet<String>(value.getAuxiliary());
      aux.remove("top");
      if (value.getStructural() != null)
      {

 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/util/SchemaLoader.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2008-2010 Sun Microsystems, Inc.
 *      Portions Copyright 2013-2014 ForgeRock AS.
 *      Portions Copyright 2013-2015 ForgeRock AS.
 */

package org.opends.guitools.controlpanel.util;
@@ -94,14 +94,8 @@
        attributesToKeep.add(attr);
      }
    }
    for (MatchingRule mr : sc.getMatchingRules().values())
    {
      matchingRulesToKeep.add(mr);
    }
    for (AttributeSyntax<?> syntax : sc.getSyntaxes().values())
    {
      syntaxesToKeep.add(syntax);
    }
    matchingRulesToKeep.addAll(sc.getMatchingRules().values());
    syntaxesToKeep.addAll(sc.getSyntaxes().values());
  }

  private static String getSchemaDirectoryPath()

 opendj-server-legacy/src/main/java/org/opends/quicksetup/installer/InstallerHelper.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2006-2010 Sun Microsystems, Inc.
 *      Portions Copyright 2011-2014 ForgeRock AS
 *      Portions Copyright 2011-2015 ForgeRock AS
 */

package org.opends.quicksetup.installer;
@@ -573,8 +573,7 @@
      {
        servers = new HashSet<String>();
      }
      Set<String> oldServers = new HashSet<String>();
      oldServers.addAll(servers);
      Set<String> oldServers = new HashSet<String>(servers);
      for (Set<String> rs : replicationServers.values())
      {
        servers.addAll(rs);
@@ -583,9 +582,7 @@
      replicationServer.setReplicationServer(servers);
      replicationServer.commit();

      Set<String> newReplicationServers = new HashSet<String>();
      newReplicationServers.addAll(servers);
      newReplicationServers.removeAll(oldServers);
      Set<String> newReplicationServers = intersect(servers, oldServers);

      /*
       * Create the domains
@@ -642,9 +639,7 @@
        usedServerIds.add(domain.getServerId());

        domain.commit();
        Set<String> addedServers = new TreeSet<String>();
        addedServers.addAll(servers);
        addedServers.removeAll(oldServers);
        Set<String> addedServers = intersect(servers, oldServers);
        ConfiguredDomain domainConf = new ConfiguredDomain(domainName,
            isCreated, addedServers);
        domainsConf.add(domainConf);
@@ -663,9 +658,16 @@
    }
  }

  private Set<String> intersect(Set<String> set1, Set<String> set2)
  {
    Set<String> result = new TreeSet<String>(set1);
    result.removeAll(set2);
    return result;
  }

  /**
   * Configures the replication on a given server.
   * @param remoteCtx the conection to the server where we want to configure
   * @param remoteCtx the connection to the server where we want to configure
   * the replication.
   * @param replConf the object describing what was configured.
   * @param serverDisplay the server display.

 opendj-server-legacy/src/main/java/org/opends/quicksetup/installer/SuffixesToReplicateOptions.java

@@ -22,6 +22,7 @@
 *
 *
 *      Copyright 2006-2010 Sun Microsystems, Inc.
 *      Portions Copyright 2015 ForgeRock AS
 */


@@ -137,10 +138,7 @@
   */
  public Set<SuffixDescriptor> getAvailableSuffixes()
  {
    LinkedHashSet<SuffixDescriptor> copy =
      new LinkedHashSet<SuffixDescriptor>();
    copy.addAll(availableSuffixes);
    return copy;
    return new LinkedHashSet<SuffixDescriptor>(availableSuffixes);
  }

  /**
@@ -151,10 +149,7 @@
   */
  public Set<SuffixDescriptor> getSuffixes()
  {
    LinkedHashSet<SuffixDescriptor> copy =
      new LinkedHashSet<SuffixDescriptor>();
    copy.addAll(suffixesToReplicate);
    return copy;
    return new LinkedHashSet<SuffixDescriptor>(suffixesToReplicate);
  }
}


 opendj-server-legacy/src/main/java/org/opends/server/backends/task/Task.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2006-2009 Sun Microsystems, Inc.
 *      Portions Copyright 2014 ForgeRock AS
 *      Portions Copyright 2014-2015 ForgeRock AS
 */
package org.opends.server.backends.task;

@@ -1156,8 +1156,7 @@
  {
    if (DirectoryServer.mailServerConfigured())
    {
      LinkedHashSet<String> recipients = new LinkedHashSet<String>();
      recipients.addAll(notifyOnCompletion);
      LinkedHashSet<String> recipients = new LinkedHashSet<String>(notifyOnCompletion);
      if (! TaskState.isSuccessful(taskState))
      {
        recipients.addAll(notifyOnError);

 opendj-server-legacy/src/main/java/org/opends/server/config/ConfigAttribute.java

@@ -22,7 +22,7 @@
 *
 *
 *      Copyright 2006-2008 Sun Microsystems, Inc.
 *      Portions Copyright 2014 ForgeRock AS
 *      Portions Copyright 2014-2015 ForgeRock AS
 */
package org.opends.server.config;
import org.forgerock.i18n.LocalizableMessage;
@@ -563,20 +563,8 @@

    // Create a temporary set of values that we will use for this change.  It
    // may not actually be applied if an error occurs for some reason.
    LinkedHashSet<ByteString> tempValues;
    if (requiresAdminAction && hasPendingValues)
    {
      tempValues =
           new LinkedHashSet<ByteString>(pendingValues.size() + numValues);
      tempValues.addAll(pendingValues);
    }
    else
    {
      tempValues =
           new LinkedHashSet<ByteString>(activeValues.size() + numValues);
      tempValues.addAll(activeValues);
    }

    final LinkedHashSet<ByteString> vals = getValues();
    LinkedHashSet<ByteString> tempValues = new LinkedHashSet<ByteString>(vals.size() + numValues);

    // Iterate through all of the provided values.  Make sure that each is
    // acceptable for use and that it is not already contained in the value set.
@@ -612,7 +600,12 @@
    }
  }


  private LinkedHashSet<ByteString> getValues()
  {
    return requiresAdminAction && hasPendingValues
        ? pendingValues
        : activeValues;
  }

  /**
   * Attempts to remove the set of values from this configuration attribute.
@@ -624,23 +617,11 @@
   *                           value set, or if this is a required attribute and
   *                           the resulting value list would be empty.
   */
  protected void removeValues(List<ByteString> values)
         throws ConfigException
  protected void removeValues(List<ByteString> values) throws ConfigException
  {
    // Create a temporary set of values that we will use for this change.  It
    // may not actually be applied if an error occurs for some reason.
    LinkedHashSet<ByteString> tempValues;
    if (requiresAdminAction && hasPendingValues)
    {
      tempValues = new LinkedHashSet<ByteString>(pendingValues.size());
      tempValues.addAll(pendingValues);
    }
    else
    {
      tempValues = new LinkedHashSet<ByteString>(activeValues.size());
      tempValues.addAll(activeValues);
    }

    LinkedHashSet<ByteString> tempValues = new LinkedHashSet<ByteString>(getValues());

    // Iterate through all the provided values and make sure that they are
    // contained in the list.  If not, then throw an exception.  If so, then

 opendj-server-legacy/src/main/java/org/opends/server/extensions/PasswordModifyExtendedOperation.java

@@ -1093,12 +1093,7 @@

  private Collection<ByteString> toAttributeValues(Collection<ByteString> values)
  {
    Set<ByteString> results = new LinkedHashSet<ByteString>(values.size());
    for (ByteString s : values)
    {
      results.add(s);
    }
    return results;
    return new LinkedHashSet<ByteString>(values);
  }

  /**

 opendj-server-legacy/src/main/java/org/opends/server/plugins/ReferentialIntegrityPlugin.java

@@ -206,19 +206,10 @@
    final ConfigChangeResult ccr = new ConfigChangeResult();

    //Load base DNs from new configuration.
    LinkedHashSet<DN> newConfiguredBaseDNs = new LinkedHashSet<DN>();
    for(DN baseDN : newConfiguration.getBaseDN())
    {
      newConfiguredBaseDNs.add(baseDN);
    }

    LinkedHashSet<DN> newConfiguredBaseDNs = new LinkedHashSet<DN>(newConfiguration.getBaseDN());
    //Load attribute types from new configuration.
    LinkedHashSet<AttributeType> newAttributeTypes =
            new LinkedHashSet<AttributeType>();
    for (AttributeType type : newConfiguration.getAttributeType())
    {
      newAttributeTypes.add(type);
    }
            new LinkedHashSet<AttributeType>(newConfiguration.getAttributeType());

    // Load the attribute-filter mapping


 opendj-server-legacy/src/main/java/org/opends/server/tools/BackUpDB.java

@@ -758,10 +758,7 @@
    {
      // Iterate through the set of backends and pick out those that were
      // requested.
      HashSet<String> requestedBackends =
           new HashSet<String>(backendList.size());
      requestedBackends.addAll(backendID.getValues());

      HashSet<String> requestedBackends = new HashSet<String>(backendID.getValues());
      for (int i=0; i < numBackends; i++)
      {
        Backend<?> b = backendList.get(i);

 opendj-server-legacy/src/main/java/org/opends/server/tools/LDAPDelete.java

@@ -679,10 +679,7 @@
    }

    ArrayList<String> trailingArgs = argParser.getTrailingArguments();
    for(String s : trailingArgs)
    {
      dnStrings.add(s);
    }
    dnStrings.addAll(trailingArgs);

    // Set the connection options.
    // Parse the SASL properties.

 opendj-server-legacy/src/main/java/org/opends/server/tools/LDAPSearch.java

@@ -1117,12 +1117,7 @@
          return CLIENT_SIDE_PARAM_ERROR;
        }
      }
      // The rest are attributes
      for(String s : filterAndAttributeStrings)
      {
        attributes.add(s);
      }

      attributes.addAll(filterAndAttributeStrings);
    }

    if(bindPassword.isPresent() && bindPasswordFile.isPresent())

 opendj-server-legacy/src/main/java/org/opends/server/util/AddChangeRecordEntry.java

@@ -82,10 +82,7 @@
    this.attributes = new ArrayList<Attribute>(attributes.size());
    for (List<Attribute> list : attributes.values())
    {
      for (Attribute a : list)
      {
        this.attributes.add(a);
      }
      this.attributes.addAll(list);
    }
  }


 opendj-server-legacy/src/test/java/org/opends/server/authorization/dseecompat/AciTests.java

@@ -834,8 +834,7 @@
  public Object[][] invalidAcis() throws Exception {
    TestCaseUtils.startServer();  // This appears to be necessary since the DataProviders can be called before @BeforeClass.

    List<String> invalid = new ArrayList<String>();
    invalid.addAll(Arrays.asList(INVALID_ACIS));
    List<String> invalid = new ArrayList<String>(Arrays.asList(INVALID_ACIS));
    for (String[] aciAndMask: INVALID_ACIS_IF_ANY_CHAR_REMOVED) {
      invalid.addAll(getAciMissingCharCombos(aciAndMask[0], aciAndMask[1]));
    }
@@ -1737,22 +1736,6 @@
      }
    }

    private List<SingleSearchParams> explodeTestParams() throws Exception {
      List<SingleSearchParams> explodedTests = new ArrayList<SingleSearchParams>();

      for (SingleSearchParams searchTest: _searchTests) {
        // Add the search test as is.
        explodedTests.add(searchTest);

        // Enabling this doubles the number of test cases without much benefit, so we disable it for now
        // And add it with the ACIs merged into the initial import
        // String ditWithAcis = applyChangesToLdif(searchTest._initialDitLdif, searchTest._aciLdif);
        // explodedTests.add(searchTest.clone(ditWithAcis, ""));
      }

      return explodedTests;
    }

    /**
     * @return the LDIF result of applying changesLdif to changesLdif
     */
@@ -1780,8 +1763,7 @@
      List<Object[]> allTestParams = new ArrayList<Object[]>();

      for (SearchTestParams testParams: SEARCH_TEST_PARAMS) {
        List<SingleSearchParams> explodedTests = testParams.explodeTestParams();
        for (SingleSearchParams singleTest: explodedTests) {
        for (SingleSearchParams singleTest: testParams._searchTests) {
          allTestParams.add(new Object[]{singleTest});
        }
      }

 opendj-server-legacy/src/test/java/org/opends/server/types/TestDN.java

@@ -36,14 +36,11 @@

import org.opends.server.TestCaseUtils;
import org.opends.server.core.DirectoryServer;
import org.opends.server.util.Platform;
import org.testng.Assert;
import org.testng.annotations.DataProvider;
import org.testng.annotations.Test;
import org.testng.annotations.BeforeClass;



/**
 * This class defines a set of tests for the org.opends.server.core.DN
 * class.
@@ -568,15 +565,8 @@
  @DataProvider(name = "namingContexts")
  public Object[][] getNamingContexts() {
    ArrayList<DN> contextList = new ArrayList<DN>();
    for (DN baseDN : DirectoryServer.getPublicNamingContexts().keySet())
    {
      contextList.add(baseDN);
    }

    for (DN baseDN : DirectoryServer.getPrivateNamingContexts().keySet())
    {
      contextList.add(baseDN);
    }
    contextList.addAll(DirectoryServer.getPublicNamingContexts().keySet());
    contextList.addAll(DirectoryServer.getPrivateNamingContexts().keySet());

    Object[][] contextArray = new Object[contextList.size()][1];
    for (int i=0; i < contextArray.length; i++)