Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 100CF200B3E for ; Wed, 7 Sep 2016 15:55:38 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 0EB25160AC1; Wed, 7 Sep 2016 13:55:38 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id B0072160AA3 for ; Wed, 7 Sep 2016 15:55:36 +0200 (CEST) Received: (qmail 87161 invoked by uid 500); 7 Sep 2016 13:55:35 -0000 Mailing-List: contact commits-help@openjpa.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@openjpa.apache.org Delivered-To: mailing list commits@openjpa.apache.org Received: (qmail 87152 invoked by uid 99); 7 Sep 2016 13:55:35 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 07 Sep 2016 13:55:35 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id 79F4D1858CD for ; Wed, 7 Sep 2016 13:55:35 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 0.374 X-Spam-Level: X-Spam-Status: No, score=0.374 tagged_above=-999 required=6.31 tests=[KAM_ASCII_DIVIDERS=0.8, KAM_LAZY_DOMAIN_SECURITY=1, RP_MATCHES_RCVD=-1.426] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id 2esqbJQnBS-C for ; Wed, 7 Sep 2016 13:55:32 +0000 (UTC) Received: from mailrelay1-us-west.apache.org (mailrelay1-us-west.apache.org [209.188.14.139]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTP id 83A955F369 for ; Wed, 7 Sep 2016 13:55:31 +0000 (UTC) Received: from svn01-us-west.apache.org (svn.apache.org [10.41.0.6]) by mailrelay1-us-west.apache.org (ASF Mail Server at mailrelay1-us-west.apache.org) with ESMTP id A6486E0099 for ; Wed, 7 Sep 2016 13:55:30 +0000 (UTC) Received: from svn01-us-west.apache.org (localhost [127.0.0.1]) by svn01-us-west.apache.org (ASF Mail Server at svn01-us-west.apache.org) with ESMTP id A4FFF3A0249 for ; Wed, 7 Sep 2016 13:55:30 +0000 (UTC) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r1759614 - in /openjpa/trunk: openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/exps/ openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/schema/ openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/ openjpa-kernel/src/main/java/or... Date: Wed, 07 Sep 2016 13:55:30 -0000 To: commits@openjpa.apache.org From: struberg@apache.org X-Mailer: svnmailer-1.0.9 Message-Id: <20160907135530.A4FFF3A0249@svn01-us-west.apache.org> archived-at: Wed, 07 Sep 2016 13:55:38 -0000 Author: struberg Date: Wed Sep 7 13:55:30 2016 New Revision: 1759614 URL: http://svn.apache.org/viewvc?rev=1759614&view=rev Log: OPENJPA-2665 use Objects.equals instead of prorietary impl Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/exps/PCPath.java openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/schema/ForeignKey.java openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/JoinSet.java openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/PrimaryRow.java openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/SQLBuffer.java openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/DelegatingDataCache.java openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/DelegatingQueryCache.java openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryKey.java openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AttachStrategy.java openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/QueryImpl.java openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/CandidatePath.java openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/InMemoryExpressionFactory.java openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/FetchGroup.java openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/util/ArrayStateImage.java openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/FileValue.java openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ObjectValue.java openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/rop/AbstractNonSequentialResultList.java openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/rop/AbstractSequentialResultList.java Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/exps/PCPath.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/exps/PCPath.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/exps/PCPath.java (original) +++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/exps/PCPath.java Wed Sep 7 13:55:30 2016 @@ -23,8 +23,8 @@ import java.sql.SQLException; import java.util.Iterator; import java.util.LinkedList; import java.util.ListIterator; +import java.util.Objects; -import org.apache.commons.lang3.ObjectUtils; import org.apache.openjpa.jdbc.kernel.JDBCFetchConfiguration; import org.apache.openjpa.jdbc.kernel.JDBCStoreQuery; import org.apache.openjpa.jdbc.meta.ClassMapping; @@ -61,10 +61,8 @@ import org.apache.openjpa.util.UserExcep * * @author Abe White */ -public class PCPath - extends CandidatePath - implements JDBCPath { - +public class PCPath extends CandidatePath implements JDBCPath { + protected static final String TRUE = "1 = 1"; protected static final String FALSE = "1 <> 1"; @@ -1112,8 +1110,8 @@ public class PCPath if (!(other instanceof PCPath)) return false; PCPath path = (PCPath) other; - return ObjectUtils.equals(_candidate, path._candidate) - && ObjectUtils.equals(_actions, path._actions); + return Objects.equals(_candidate, path._candidate) + && Objects.equals(_actions, path._actions); } public int getId() { @@ -1158,7 +1156,7 @@ public class PCPath return true; Action a = (Action) other; return op == a.op - && ObjectUtils.equals(data, a.data); + && Objects.equals(data, a.data); } } } Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/schema/ForeignKey.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/schema/ForeignKey.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/schema/ForeignKey.java (original) +++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/schema/ForeignKey.java Wed Sep 7 13:55:30 2016 @@ -24,8 +24,8 @@ import java.util.ArrayList; import java.util.Arrays; import java.util.LinkedHashMap; import java.util.List; +import java.util.Objects; -import org.apache.commons.lang3.ObjectUtils; import org.apache.openjpa.jdbc.identifier.DBIdentifier; import org.apache.openjpa.jdbc.sql.DBDictionary; import org.apache.openjpa.lib.log.Log; @@ -558,7 +558,7 @@ public class ForeignKey * Join a local column to a primary key column of another table. */ public void join(Column local, Column toPK) { - if (!ObjectUtils.equals(local.getTable(), getTable())) + if (!Objects.equals(local.getTable(), getTable())) throw new InvalidStateException(_loc.get("table-mismatch", local.getTable(), getTable())); @@ -776,7 +776,7 @@ public class ForeignKey if (vals.length != fkVals.length) return false; for (int i = 0; i < vals.length; i++) - if (!ObjectUtils.equals(vals[i], fkVals[i])) + if (!Objects.equals(vals[i], fkVals[i])) return false; return true; } Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/JoinSet.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/JoinSet.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/JoinSet.java (original) +++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/JoinSet.java Wed Sep 7 13:55:30 2016 @@ -25,7 +25,7 @@ import java.util.LinkedList; import java.util.List; import java.util.NoSuchElementException; -import org.apache.commons.lang3.ObjectUtils; +import java.util.Objects; /** * Set type that recognizes that inner joins should take precedence @@ -382,8 +382,8 @@ class JoinSet { if (!(other instanceof Node)) return false; Node node = (Node) other; - return ObjectUtils.equals(join, node.join) - && ObjectUtils.equals(next, node.next); + return Objects.equals(join, node.join) + && Objects.equals(next, node.next); } public Object clone() { Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/PrimaryRow.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/PrimaryRow.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/PrimaryRow.java (original) +++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/PrimaryRow.java Wed Sep 7 13:55:30 2016 @@ -20,7 +20,7 @@ package org.apache.openjpa.jdbc.sql; import java.sql.SQLException; -import org.apache.commons.lang3.ObjectUtils; +import java.util.Objects; import org.apache.openjpa.jdbc.meta.ClassMapping; import org.apache.openjpa.jdbc.meta.RelationId; import org.apache.openjpa.jdbc.schema.Column; @@ -380,7 +380,7 @@ public class PrimaryRow * Return true if the two values should be considered equal. */ private static boolean rowValueEquals(Object o1, Object o2) { - if (ObjectUtils.equals(o1, o2)) + if (Objects.equals(o1, o2)) return true; // check for numeric equality (bug #1151) Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/SQLBuffer.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/SQLBuffer.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/SQLBuffer.java (original) +++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/sql/SQLBuffer.java Wed Sep 7 13:55:30 2016 @@ -29,7 +29,7 @@ import java.util.Collections; import java.util.Iterator; import java.util.List; -import org.apache.commons.lang3.ObjectUtils; +import java.util.Objects; import org.apache.openjpa.jdbc.identifier.DBIdentifier; import org.apache.openjpa.jdbc.kernel.JDBCFetchConfiguration; import org.apache.openjpa.jdbc.kernel.exps.CollectionParam; @@ -630,7 +630,7 @@ public final class SQLBuffer SQLBuffer buf = (SQLBuffer) other; return _sql.equals(buf._sql) - && ObjectUtils.equals(_params, buf._params); + && Objects.equals(_params, buf._params); } /** Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/DelegatingDataCache.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/DelegatingDataCache.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/DelegatingDataCache.java (original) +++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/DelegatingDataCache.java Wed Sep 7 13:55:30 2016 @@ -24,7 +24,7 @@ import java.util.List; import java.util.Map; import java.util.Set; -import org.apache.commons.lang3.ObjectUtils; +import java.util.Objects; import org.apache.openjpa.util.RuntimeExceptionTranslator; /** @@ -85,7 +85,7 @@ public class DelegatingDataCache return true; if (other instanceof DelegatingDataCache) other = ((DelegatingDataCache) other).getInnermostDelegate(); - return ObjectUtils.equals(getInnermostDelegate(), other); + return Objects.equals(getInnermostDelegate(), other); } /** Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/DelegatingQueryCache.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/DelegatingQueryCache.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/DelegatingQueryCache.java (original) +++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/DelegatingQueryCache.java Wed Sep 7 13:55:30 2016 @@ -18,7 +18,7 @@ */ package org.apache.openjpa.datacache; -import org.apache.commons.lang3.ObjectUtils; +import java.util.Objects; import org.apache.openjpa.kernel.QueryStatistics; import org.apache.openjpa.util.RuntimeExceptionTranslator; @@ -77,7 +77,7 @@ public class DelegatingQueryCache return true; if (other instanceof DelegatingQueryCache) other = ((DelegatingQueryCache) other).getInnermostDelegate(); - return ObjectUtils.equals(getInnermostDelegate(), other); + return Objects.equals(getInnermostDelegate(), other); } /** Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryKey.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryKey.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryKey.java (original) +++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/datacache/QueryKey.java Wed Sep 7 13:55:30 2016 @@ -36,7 +36,7 @@ import java.util.Set; import java.util.SortedSet; import java.util.TreeSet; -import org.apache.commons.lang3.ObjectUtils; +import java.util.Objects; import org.apache.commons.lang3.StringUtils; import org.apache.openjpa.enhance.PCRegistry; import org.apache.openjpa.kernel.Query; @@ -408,8 +408,8 @@ public class QueryKey && _ignoreChanges == other._ignoreChanges && _rangeStart == other._rangeStart && _rangeEnd == other._rangeEnd - && ObjectUtils.equals(_query, other._query) - && ObjectUtils.equals(_params, other._params); + && Objects.equals(_query, other._query) + && Objects.equals(_params, other._params); } /** Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AttachStrategy.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AttachStrategy.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AttachStrategy.java (original) +++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AttachStrategy.java Wed Sep 7 13:55:30 2016 @@ -25,7 +25,7 @@ import java.util.Map; import java.util.Set; import java.util.Map.Entry; -import org.apache.commons.lang3.ObjectUtils; +import java.util.Objects; import org.apache.openjpa.enhance.PersistenceCapable; import org.apache.openjpa.enhance.StateManager; import org.apache.openjpa.lib.util.Localizer; @@ -225,7 +225,7 @@ abstract class AttachStrategy else { PersistenceCapable intopc = topc; if (!fmd.isEmbeddedPC() && frmpc != null && topc != null - && !ObjectUtils.equals(topc.pcFetchObjectId(), + && !Objects.equals(topc.pcFetchObjectId(), manager.getDetachedObjectId(frmpc))) { intopc = null; } Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/QueryImpl.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/QueryImpl.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/QueryImpl.java (original) +++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/QueryImpl.java Wed Sep 7 13:55:30 2016 @@ -33,7 +33,7 @@ import java.util.Map.Entry; import java.util.concurrent.locks.ReentrantLock; import org.apache.commons.collections.map.LinkedMap; -import org.apache.commons.lang3.ObjectUtils; +import java.util.Objects; import org.apache.commons.lang3.StringUtils; import org.apache.openjpa.conf.OpenJPAConfiguration; import org.apache.openjpa.enhance.PersistenceCapable; @@ -1848,7 +1848,7 @@ public class QueryImpl return false; if (key.subclasses != subclasses) return false; - if (!ObjectUtils.equals(key.storeKey, storeKey)) + if (!Objects.equals(key.storeKey, storeKey)) return false; // allow either candidate type to be null because it might be Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/CandidatePath.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/CandidatePath.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/CandidatePath.java (original) +++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/CandidatePath.java Wed Sep 7 13:55:30 2016 @@ -22,7 +22,7 @@ import java.util.Iterator; import java.util.LinkedList; import java.util.ListIterator; -import org.apache.commons.lang3.ObjectUtils; +import java.util.Objects; import org.apache.openjpa.kernel.Broker; import org.apache.openjpa.kernel.Filters; import org.apache.openjpa.kernel.OpenJPAStateManager; @@ -161,7 +161,7 @@ public class CandidatePath return true; if (!(other instanceof CandidatePath)) return false; - return ObjectUtils.equals(_actions, ((CandidatePath) other)._actions); + return Objects.equals(_actions, ((CandidatePath) other)._actions); } /** Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/InMemoryExpressionFactory.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/InMemoryExpressionFactory.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/InMemoryExpressionFactory.java (original) +++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/exps/InMemoryExpressionFactory.java Wed Sep 7 13:55:30 2016 @@ -30,7 +30,7 @@ import java.util.List; import java.util.ListIterator; import java.util.Set; -import org.apache.commons.lang3.ObjectUtils; +import java.util.Objects; import org.apache.openjpa.kernel.Extent; import org.apache.openjpa.kernel.StoreContext; import org.apache.openjpa.lib.util.Localizer; @@ -140,7 +140,7 @@ public class InMemoryExpressionFactory for (int i = 0; i < exps.grouping.length; i++) { curs[i] = ((Val) exps.grouping[i]).evaluate(pc, pc, ctx, params); - eq = eq && ObjectUtils.equals(prevs[i], curs[i]); + eq = eq && Objects.equals(prevs[i], curs[i]); } // if this object's grouping values differ from the prev, @@ -695,7 +695,7 @@ public class InMemoryExpressionFactory if (_arr.length != arr.length) return false; for (int i = 0; i < _arr.length; i++) - if (!ObjectUtils.equals(_arr[i], arr[i])) + if (!Objects.equals(_arr[i], arr[i])) return false; return true; } Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/FetchGroup.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/FetchGroup.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/FetchGroup.java (original) +++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/FetchGroup.java Wed Sep 7 13:55:30 2016 @@ -28,7 +28,7 @@ import java.util.Map; import java.util.Set; import org.apache.commons.lang3.StringUtils; -import org.apache.commons.lang3.ObjectUtils; +import java.util.Objects; import org.apache.openjpa.lib.util.Localizer; import org.apache.openjpa.util.MetaDataException; @@ -375,7 +375,7 @@ public class FetchGroup return false; FetchGroup that = (FetchGroup) other; return _name.equals(that._name) - && ObjectUtils.equals(_meta, that._meta); + && Objects.equals(_meta, that._meta); } public int hashCode() { Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/util/ArrayStateImage.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/util/ArrayStateImage.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/util/ArrayStateImage.java (original) +++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/util/ArrayStateImage.java Wed Sep 7 13:55:30 2016 @@ -19,8 +19,9 @@ package org.apache.openjpa.util; import java.util.BitSet; +import java.util.Objects; -import org.apache.commons.lang3.ObjectUtils; +import java.util.Objects; /** * Utilities for dealing with a simple state image consisting of an @@ -98,7 +99,7 @@ public class ArrayStateImage { BitSet loaded2 = getLoaded(state2); for (int i = 0, max = loaded1.length(); i < max; i++) { if (loaded1.get(i) && loaded2.get(i) - && !ObjectUtils.equals(state1[i], state2[i])) + && !Objects.equals(state1[i], state2[i])) return false; } return true; Modified: openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/FileValue.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/FileValue.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/FileValue.java (original) +++ openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/FileValue.java Wed Sep 7 13:55:30 2016 @@ -20,8 +20,8 @@ package org.apache.openjpa.lib.conf; import java.io.File; import java.security.AccessController; +import java.util.Objects; -import org.apache.commons.lang3.ObjectUtils; import org.apache.openjpa.lib.util.J2DoPrivHelper; /** @@ -48,7 +48,7 @@ public class FileValue extends Value { assertChangeable(); File oldValue = this.value; this.value = value; - if (!ObjectUtils.equals(oldValue, value)) + if (!Objects.equals(oldValue, value)) valueChanged(); } Modified: openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ObjectValue.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ObjectValue.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ObjectValue.java (original) +++ openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ObjectValue.java Wed Sep 7 13:55:30 2016 @@ -19,8 +19,8 @@ package org.apache.openjpa.lib.conf; import java.security.AccessController; +import java.util.Objects; -import org.apache.commons.lang3.ObjectUtils; import org.apache.openjpa.lib.util.J2DoPrivHelper; import org.apache.openjpa.lib.util.Localizer; import org.apache.openjpa.lib.util.ReferenceMap; @@ -69,7 +69,7 @@ public class ObjectValue extends Value { if (!derived) assertChangeable(); Object oldValue = _value; _value = obj; - if (!derived && !ObjectUtils.equals(obj, oldValue)) { + if (!derived && !Objects.equals(obj, oldValue)) { objectChanged(); valueChanged(); } Modified: openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/rop/AbstractNonSequentialResultList.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/rop/AbstractNonSequentialResultList.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/rop/AbstractNonSequentialResultList.java (original) +++ openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/rop/AbstractNonSequentialResultList.java Wed Sep 7 13:55:30 2016 @@ -25,8 +25,8 @@ import java.util.Iterator; import java.util.List; import java.util.ListIterator; import java.util.NoSuchElementException; +import java.util.Objects; -import org.apache.commons.lang3.ObjectUtils; /** * Abstract base class for random-access result lists. Unlike the @@ -53,7 +53,7 @@ public abstract class AbstractNonSequent obj = getInternal(i); if (obj == PAST_END) break; - if (ObjectUtils.equals(o, obj)) + if (Objects.equals(o, obj)) return true; } return false; @@ -82,7 +82,7 @@ public abstract class AbstractNonSequent obj = getInternal(i); if (obj == PAST_END) break; - if (ObjectUtils.equals(o, obj)) + if (Objects.equals(o, obj)) return i; } return -1; @@ -96,7 +96,7 @@ public abstract class AbstractNonSequent obj = getInternal(i); if (obj == PAST_END) break; - if (ObjectUtils.equals(o, obj)) + if (Objects.equals(o, obj)) index = i; } return index; Modified: openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/rop/AbstractSequentialResultList.java URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/rop/AbstractSequentialResultList.java?rev=1759614&r1=1759613&r2=1759614&view=diff ============================================================================== --- openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/rop/AbstractSequentialResultList.java (original) +++ openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/rop/AbstractSequentialResultList.java Wed Sep 7 13:55:30 2016 @@ -23,8 +23,8 @@ import java.util.ArrayList; import java.util.Collection; import java.util.Iterator; import java.util.ListIterator; +import java.util.Objects; -import org.apache.commons.lang3.ObjectUtils; /** * Abstract base class for sequential result lists. Unlike the @@ -43,7 +43,7 @@ public abstract class AbstractSequential public boolean contains(Object o) { assertOpen(); for (Iterator itr = itr(0); itr.hasNext();) - if (ObjectUtils.equals(o, itr.next())) + if (Objects.equals(o, itr.next())) return true; return false; } @@ -65,7 +65,7 @@ public abstract class AbstractSequential assertOpen(); int index = 0; for (Iterator itr = itr(0); itr.hasNext(); index++) - if (ObjectUtils.equals(o, itr.next())) + if (Objects.equals(o, itr.next())) return index; return -1; } @@ -75,7 +75,7 @@ public abstract class AbstractSequential int index = -1; int i = 0; for (Iterator itr = itr(0); itr.hasNext(); i++) - if (ObjectUtils.equals(o, itr.next())) + if (Objects.equals(o, itr.next())) index = i; return index; }