OPENDJ-1666 code cleanup

Gaetan Boismal

17.45.2015 4a438e182fd5349dc4a36ca59fc3c3501442f49d

OPENDJ-1666 code cleanup

* org/forgerock/opendj/ldap/spi/LdapPromiseWrapper.java
** Improve consistency of imports

* clirr-ignored-api-changes.xml
** Format file

 opendj-core/clirr-ignored-api-changes.xml

@@ -432,13 +432,12 @@
    <method>boolean startsWith(org.forgerock.opendj.ldap.ByteSequence)</method>
    <justification>Lack of startsWith() forced to re-implement it multiple times at different location</justification>
  </difference>
    <difference>
        <className>org/forgerock/opendj/ldap/ByteString</className>
        <differenceType>7005</differenceType>
        <method>org.forgerock.opendj.ldap.ByteString valueOf(java.lang.String)
        </method>
        <from>org.forgerock.opendj.ldap.ByteString valueOf(java.lang.String)</from>
        <to>org.forgerock.opendj.ldap.ByteString valueOf(java.lang.CharSequence)</to>
        <justification>Using CharSequence instead of String allows to reduce memory copy.</justification>
    </difference>
  <difference>
    <className>org/forgerock/opendj/ldap/ByteString</className>
    <differenceType>7005</differenceType>
    <method>org.forgerock.opendj.ldap.ByteString valueOf(java.lang.String)</method>
    <from>org.forgerock.opendj.ldap.ByteString valueOf(java.lang.String)</from>
    <to>org.forgerock.opendj.ldap.ByteString valueOf(java.lang.CharSequence)</to>
    <justification>Using CharSequence instead of String allows to reduce memory copy.</justification>
  </difference>
</differences>

 opendj-core/src/main/java/org/forgerock/opendj/ldap/spi/LdapPromiseWrapper.java

@@ -134,7 +134,7 @@
    @Override
    // @Checkstyle:ignore
    public <VOUT> LdapPromise<VOUT> then(Function<? super R, VOUT, LdapException> onSuccess) {
        return LdapPromises.wrap(wrappedPromise.then(onSuccess), getRequestID());
        return wrap(wrappedPromise.then(onSuccess), getRequestID());
    }

    @Override