Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
MIME-Version: 1.0
In-Reply-To: 
 <CAB917RKihWphcZagGh1vv3PBqm+RKE8OwU1dCwLf-UuiwO6TKA@mail.gmail.com>
References: <20131014181540.1FED62388868@eris.apache.org>
	<CAE9L6G15+qW3EmXz=b50Hnk6iJ9FT=BQx+tCV9S28xCu5kRQAQ@mail.gmail.com>
	<CAB917RKihWphcZagGh1vv3PBqm+RKE8OwU1dCwLf-UuiwO6TKA@mail.gmail.com>
Date: Mon, 14 Oct 2013 20:36:31 +0200
Message-ID: 
 <CAB917RJYADmTWwLcuF4qQU8+o8BFOt1mezkgZO9dr-xFg=e_fA@mail.gmail.com>
Subject: Re: svn commit: r1532011 - in
 /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3:
 ArrayUtils.java ObjectUtils.java
From: Benedikt Ritter <britter@apache.org>
To: Commons Developers List <dev@commons.apache.org>
Content-Type: multipart/alternative; boundary=047d7bf0d6283290da04e8b7bf41

--047d7bf0d6283290da04e8b7bf41
Content-Type: text/plain; charset=ISO-8859-1

Hi Matt,

(fired the last one without adding my comment :-)


2013/10/14 Benedikt Ritter <beneritter@gmail.com>

>
>
>
> 2013/10/14 Matt Benson <gudnabrsam@gmail.com>
>
>> Hi Benedikt, see inline:
>>
>>
>> On Mon, Oct 14, 2013 at 1:15 PM, <britter@apache.org> wrote:
>>
>> > Author: britter
>> > Date: Mon Oct 14 18:15:39 2013
>> > New Revision: 1532011
>> >
>> > URL: http://svn.apache.org/r1532011
>> > Log:
>> > Deprecate methods that are available in Java 7's java.lang.Objects
>> >
>> > Modified:
>> >
>> >
>> commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ArrayUtils.java
>> >
>> >
>> commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ObjectUtils.java
>> >
>> > Modified:
>> >
>> commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ArrayUtils.java
>> > URL:
>> >
>> http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ArrayUtils.java?rev=1532011&r1=1532010&r2=1532011&view=diff
>> >
>> >
>> ==============================================================================
>> > ---
>> >
>> commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ArrayUtils.java
>> > (original)
>> > +++
>> >
>> commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ArrayUtils.java
>> > Mon Oct 14 18:15:39 2013
>> > @@ -199,6 +199,8 @@ public class ArrayUtils {
>> >       * @param array1  the left hand array to compare, may be {@code
>> null}
>> >       * @param array2  the right hand array to compare, may be {@code
>> null}
>> >       * @return {@code true} if the arrays are equal
>> > +     * @deprecated this method has been replaced by {@code
>> > java.util.Objects.deepEquals(Object, Object)} and will be
>> > +     * removed from future releases.
>> >       */
>> >      public static boolean isEquals(final Object array1, final Object
>> > array2) {
>> >          return new EqualsBuilder().append(array1, array2).isEquals();
>> >
>> > Modified:
>> >
>> commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ObjectUtils.java
>> > URL:
>> >
>> http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ObjectUtils.java?rev=1532011&r1=1532010&r2=1532011&view=diff
>> >
>> >
>> ==============================================================================
>> > ---
>> >
>> commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ObjectUtils.java
>> > (original)
>> > +++
>> >
>> commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ObjectUtils.java
>> > Mon Oct 14 18:15:39 2013
>> > @@ -149,6 +149,8 @@ public class ObjectUtils {
>> >       * @param object1  the first object, may be {@code null}
>> >       * @param object2  the second object, may be {@code null}
>> >       * @return {@code true} if the values of both objects are the same
>> > +     * @deprecated this method has been replaces by {@code
>> > java.util.Objects.equals(Object, Object)} in Java 7 and will
>> > +     * be removed from future releases.
>> >       */
>> >      public static boolean equals(final Object object1, final Object
>> > object2) {
>> >          if (object1 == object2) {
>> > @@ -195,6 +197,8 @@ public class ObjectUtils {
>> >       * @param obj  the object to obtain the hash code of, may be {@code
>> > null}
>> >       * @return the hash code of the object, or zero if null
>> >       * @since 2.1
>> > +     * @deprecated this method has been replaced by {@code
>> > java.util.Objects.hashCode(Object)} in Java 7 and will be
>> > +     * removed in future releases
>> >       */
>> >      public static int hashCode(final Object obj) {
>> >          // hashCode(Object) retained for performance, as hash code is
>> > often critical
>> > @@ -220,6 +224,8 @@ public class ObjectUtils {
>> >       * @param objects  the objects to obtain the hash code of, may be
>> > {@code null}
>> >       * @return the hash code of the objects, or zero if null
>> >       * @since 3.0
>> > +     * @deprecated this method has been replaced by {@code
>> > java.util.Objects.hash(Object...)} in Java 7 an will be
>> > +     * removed in future releases.
>> >       */
>> >      public static int hashCodeMulti(final Object... objects) {
>> >          int hash = 1;
>> > @@ -373,6 +379,9 @@ public class ObjectUtils {
>> >       * @param obj  the Object to {@code toString}, may be null
>> >       * @return the passed in Object's toString, or {@code ""} if {@code
>> > null} input
>> >       * @since 2.0
>> > +     * @deprecated this method has been replaces by {@code
>> > java.util.Objects.toString(Object)} in Java 7 and will be
>> > +     * removed in future releases. Note however that said method will
>> > return "null" for null references, while this
>> > +     * method returns and empty String. To preserve behavior use {@code
>> > java.util.Objects.toString(myObject, "")}
>> >
>>
>> My preference here would be to begin providing
>> ObjectUtils#defaultString(Object) with the existing "", intended to
>> survive
>> beyond the removal of ObjectUtils.toString().  This will:
>>  * preserve the users' ability to call a method that implicitly uses ""
>>  * reduce confusion with Objects.toString(), and
>>  * enforce mnemonic retention by using the same terminology/behavior as
>> StringUtils#defaultString()
>>
>> I'd welcome assenting or dissenting opinions here from other committers
>> and
>> users.
>>
>
Makes sense to me. So if nobody objects, feel free to change it like that.


>
>> Matt
>>
>>
>> >       */
>> >      public static String toString(final Object obj) {
>> >          return obj == null ? "" : obj.toString();
>> > @@ -396,6 +405,8 @@ public class ObjectUtils {
>> >       * @param nullStr  the String to return if {@code null} input, may
>> be
>> > null
>> >       * @return the passed in Object's toString, or {@code nullStr} if
>> > {@code null} input
>> >       * @since 2.0
>> > +     * @deprecated this method has been replaces by {@code
>> > java.util.Objects.toString(Object, String)} in Java 7 and
>> > +     * will be removed in future releases.
>> >       */
>> >      public static String toString(final Object obj, final String
>> nullStr)
>> > {
>> >          return obj == null ? nullStr : obj.toString();
>> >
>> >
>> >
>>
>
>


-- 
http://people.apache.org/~britter/
http://www.systemoutprintln.de/
http://twitter.com/BenediktRitter
http://github.com/britter

--047d7bf0d6283290da04e8b7bf41--