Fix OPENDJ-597: Remove superfluous ByteString based constructors

Matthew Swift

21.32.2012 0e52d359d661d2fd95fa5c28058353f5d121173c

 opendj3/opendj-ldap-sdk/src/main/java/org/forgerock/opendj/ldap/LinkedAttribute.java

@@ -636,27 +636,6 @@
     * {@inheritDoc}
     */
    @Override
    public boolean addAll(final Collection<? extends ByteString> values,
            final Collection<? super ByteString> duplicateValues) {
        Validator.ensureNotNull(values);

        // TODO: could optimize if objects is a LinkedAttribute having the same
        // equality matching rule.
        boolean modified = false;
        for (final ByteString value : values) {
            if (add(value)) {
                modified = true;
            } else if (duplicateValues != null) {
                duplicateValues.add(value);
            }
        }
        return modified;
    }

    /**
     * {@inheritDoc}
     */
    @Override
    public void clear() {
        pimpl.clear(this);
    }
@@ -716,27 +695,6 @@
     * {@inheritDoc}
     */
    @Override
    public <T> boolean removeAll(final Collection<T> values,
            final Collection<? super T> missingValues) {
        Validator.ensureNotNull(values);

        // TODO: could optimize if objects is a LinkedAttribute having the same
        // equality matching rule.
        boolean modified = false;
        for (final T value : values) {
            if (remove(ByteString.valueOf(value))) {
                modified = true;
            } else if (missingValues != null) {
                missingValues.add(value);
            }
        }
        return modified;
    }

    /**
     * {@inheritDoc}
     */
    @Override
    public <T> boolean retainAll(final Collection<T> values,
            final Collection<? super T> missingValues) {
        Validator.ensureNotNull(values);