xmpp-addr: Backfill missing class method for Java 1.7

This backfills missing class methods for `java.nio.charset.StandardCharsets`
and `java.util.Objects` for compatibility with platforms which do not support
these (mainly Android SDK versions <= 18).
This commit is contained in:
Alex Palaistras 2018-12-08 19:45:02 +00:00
parent ceaa3135ab
commit 69ca58d0db
3 changed files with 34 additions and 20 deletions

View file

@ -25,7 +25,7 @@
package rocks.xmpp.addr; package rocks.xmpp.addr;
import java.text.Collator; import java.text.Collator;
import java.util.Objects; import java.util.Arrays;
/** /**
* Abstract Jid implementation for both full and bare JIDs. * Abstract Jid implementation for both full and bare JIDs.
@ -75,14 +75,14 @@ abstract class AbstractJid implements Jid {
} }
Jid other = (Jid) o; Jid other = (Jid) o;
return Objects.equals(getLocal(), other.getLocal()) return (getLocal() == other.getLocal() || getLocal() != null && getLocal().equals(other.getLocal()))
&& Objects.equals(getDomain(), other.getDomain()) && (getDomain() == other.getDomain() || getDomain() != null && getDomain().equals(other.getDomain()))
&& Objects.equals(getResource(), other.getResource()); && (getResource() == other.getResource() || getResource() != null && getResource().equals(other.getResource()));
} }
@Override @Override
public final int hashCode() { public final int hashCode() {
return Objects.hash(getLocal(), getDomain(), getResource()); return Arrays.hashCode(new String[]{getLocal(), getDomain(), getResource()});
} }
/** /**

View file

@ -29,10 +29,9 @@ import rocks.xmpp.precis.PrecisProfiles;
import rocks.xmpp.util.cache.LruCache; import rocks.xmpp.util.cache.LruCache;
import java.net.IDN; import java.net.IDN;
import java.nio.charset.StandardCharsets; import java.nio.charset.Charset;
import java.text.Normalizer; import java.text.Normalizer;
import java.util.Map; import java.util.Map;
import java.util.Objects;
import java.util.regex.Matcher; import java.util.regex.Matcher;
import java.util.regex.Pattern; import java.util.regex.Pattern;
@ -109,6 +108,10 @@ final class FullJid extends AbstractJid {
final String unescapedLocalPart; final String unescapedLocalPart;
if (domain == null) {
throw new NullPointerException();
}
if (doUnescape) { if (doUnescape) {
unescapedLocalPart = unescape(local); unescapedLocalPart = unescape(local);
} else { } else {
@ -126,7 +129,7 @@ final class FullJid extends AbstractJid {
// character MUST be stripped before any other canonicalization steps // character MUST be stripped before any other canonicalization steps
// are taken. // are taken.
// Also validate, that the domain name can be converted to ASCII, i.e. validate the domain name (e.g. must not start with "_"). // Also validate, that the domain name can be converted to ASCII, i.e. validate the domain name (e.g. must not start with "_").
final String strDomain = IDN.toASCII(LABEL_SEPARATOR_FINAL.matcher(Objects.requireNonNull(domain)).replaceAll(""), IDN.USE_STD3_ASCII_RULES); final String strDomain = IDN.toASCII(LABEL_SEPARATOR_FINAL.matcher(domain).replaceAll(""), IDN.USE_STD3_ASCII_RULES);
enforcedLocalPart = escapedLocalPart != null ? PrecisProfiles.USERNAME_CASE_MAPPED.enforce(escapedLocalPart) : null; enforcedLocalPart = escapedLocalPart != null ? PrecisProfiles.USERNAME_CASE_MAPPED.enforce(escapedLocalPart) : null;
enforcedResource = resource != null ? PrecisProfiles.OPAQUE_STRING.enforce(resource) : null; enforcedResource = resource != null ? PrecisProfiles.OPAQUE_STRING.enforce(resource) : null;
// See https://tools.ietf.org/html/rfc5895#section-2 // See https://tools.ietf.org/html/rfc5895#section-2
@ -152,7 +155,7 @@ final class FullJid extends AbstractJid {
@Override @Override
public Jid withLocal(CharSequence local) { public Jid withLocal(CharSequence local) {
if (Objects.equals(local, this.getLocal())) { if (local == this.getLocal() || local != null && local.equals(this.getLocal())) {
return this; return this;
} }
return new FullJid(local, getDomain(), getResource(), false, null); return new FullJid(local, getDomain(), getResource(), false, null);
@ -160,7 +163,7 @@ final class FullJid extends AbstractJid {
@Override @Override
public Jid withResource(CharSequence resource) { public Jid withResource(CharSequence resource) {
if (Objects.equals(resource, this.getResource())) { if (resource == this.getResource() || resource != null && resource.equals(this.getResource())) {
return this; return this;
} }
return new FullJid(getLocal(), getDomain(), resource, false, asBareJid()); return new FullJid(getLocal(), getDomain(), resource, false, asBareJid());
@ -168,7 +171,10 @@ final class FullJid extends AbstractJid {
@Override @Override
public Jid atSubdomain(CharSequence subdomain) { public Jid atSubdomain(CharSequence subdomain) {
return new FullJid(getLocal(), Objects.requireNonNull(subdomain) + "." + getDomain(), getResource(), false, null); if (subdomain == null) {
throw new NullPointerException();
}
return new FullJid(getLocal(), subdomain + "." + getDomain(), getResource(), false, null);
} }
@Override @Override
@ -206,7 +212,9 @@ final class FullJid extends AbstractJid {
* @see <a href="https://xmpp.org/extensions/xep-0106.html">XEP-0106: JID Escaping</a> * @see <a href="https://xmpp.org/extensions/xep-0106.html">XEP-0106: JID Escaping</a>
*/ */
static Jid of(String jid, final boolean doUnescape) { static Jid of(String jid, final boolean doUnescape) {
Objects.requireNonNull(jid, "jid must not be null."); if (jid == null) {
throw new NullPointerException("jid must not be null.");
}
jid = jid.trim(); jid = jid.trim();
@ -278,7 +286,9 @@ final class FullJid extends AbstractJid {
} }
private static void validateDomain(String domain) { private static void validateDomain(String domain) {
Objects.requireNonNull(domain, "domain must not be null."); if (domain == null) {
throw new NullPointerException("domain must not be null.");
}
if (domain.contains("@")) { if (domain.contains("@")) {
// Prevent misuse of API. // Prevent misuse of API.
throw new IllegalArgumentException("domain must not contain a '@' sign"); throw new IllegalArgumentException("domain must not contain a '@' sign");
@ -297,7 +307,7 @@ final class FullJid extends AbstractJid {
if (value.length() == 0) { if (value.length() == 0) {
throw new IllegalArgumentException(part + " must not be empty."); throw new IllegalArgumentException(part + " must not be empty.");
} }
if (value.toString().getBytes(StandardCharsets.UTF_8).length > 1023) { if (value.toString().getBytes(Charset.forName("UTF-8")).length > 1023) {
throw new IllegalArgumentException(part + " must not be greater than 1023 bytes."); throw new IllegalArgumentException(part + " must not be greater than 1023 bytes.");
} }
} }
@ -391,7 +401,7 @@ final class FullJid extends AbstractJid {
*/ */
@Override @Override
public final Jid withLocal(CharSequence local) { public final Jid withLocal(CharSequence local) {
if (Objects.equals(local, this.getLocal())) { if (local == this.getLocal() || local != null && local.equals(this.getLocal())) {
return this; return this;
} }
return new FullJid(local, getDomain(), getResource(), false, null); return new FullJid(local, getDomain(), getResource(), false, null);
@ -408,7 +418,7 @@ final class FullJid extends AbstractJid {
*/ */
@Override @Override
public final Jid withResource(CharSequence resource) { public final Jid withResource(CharSequence resource) {
if (Objects.equals(resource, this.getResource())) { if (resource == this.getResource() || resource != null && resource.equals(this.getResource())) {
return this; return this;
} }
return new FullJid(getLocal(), getDomain(), resource, false, asBareJid()); return new FullJid(getLocal(), getDomain(), resource, false, asBareJid());
@ -424,7 +434,10 @@ final class FullJid extends AbstractJid {
*/ */
@Override @Override
public final Jid atSubdomain(CharSequence subdomain) { public final Jid atSubdomain(CharSequence subdomain) {
return new FullJid(getLocal(), Objects.requireNonNull(subdomain) + "." + getDomain(), getResource(), false, null); if (subdomain != null) {
throw new NullPointerException();
}
return new FullJid(getLocal(), subdomain + "." + getDomain(), getResource(), false, null);
} }
/** /**

View file

@ -24,8 +24,6 @@
package rocks.xmpp.addr; package rocks.xmpp.addr;
import java.util.Objects;
/** /**
* Represents a malformed JID in order to handle the <code>jid-malformed</code> error. * Represents a malformed JID in order to handle the <code>jid-malformed</code> error.
* <p> * <p>
@ -96,7 +94,10 @@ public final class MalformedJid extends AbstractJid {
@Override @Override
public Jid atSubdomain(CharSequence subdomain) { public Jid atSubdomain(CharSequence subdomain) {
return new MalformedJid(localPart, Objects.requireNonNull(subdomain) + "." + domainPart, resourcePart, cause); if (subdomain == null) {
throw new NullPointerException();
}
return new MalformedJid(localPart, subdomain + "." + domainPart, resourcePart, cause);
} }
@Override @Override