OFBiz › OFBiz - Dev
Login  Register

Re: svn commit: r1527212 - /ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilMisc.java

‹ Previous Topic Next Topic ›
 
classic Classic list List threaded Threaded
♦
♦
 2 messages Options Options
 
Embed post
Permalink
Varun Bhansaly
Reply | Threaded
Open this post in threaded view
♦
♦
| More
Print post
Permalink
Sep 29, 2013; 3:44am

Re: svn commit: r1527212 - /ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilMisc.java

Varun Bhansaly
155 posts
Hi Adrian,

This will need to be backported to release 13.07.
We should even consider creating few test cases that may help in such
refactoring exercises.

--
Regards,
Varun Bhansaly


On Sat, Sep 28, 2013 at 11:54 PM, <[hidden email]> wrote:

> Author: adrianc
> Date: Sat Sep 28 18:24:53 2013
> New Revision: 1527212
>
> URL: http://svn.apache.org/r1527212
> Log:
> Replaced HashSet with LinkedHashSet in UtilMisc.java to preserve original
> ordering. Suggested by Leon on the dev mailing list.
>
> Modified:
>     ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilMisc.java
>
> Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilMisc.java
> URL:
> http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilMisc.java?rev=1527212&r1=1527211&r2=1527212&view=diff
>
> ==============================================================================
> --- ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilMisc.java
> (original)
> +++ ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilMisc.java Sat
> Sep 28 18:24:53 2013
> @@ -30,8 +30,8 @@ import java.util.ArrayList;
>  import java.util.Collection;
>  import java.util.Collections;
>  import java.util.HashMap;
> -import java.util.HashSet;
>  import java.util.Iterator;
> +import java.util.LinkedHashSet;
>  import java.util.LinkedList;
>  import java.util.List;
>  import java.util.Locale;
> @@ -159,7 +159,7 @@ public class UtilMisc {
>              Debug.logInfo(e, module);
>              throw e;
>          }
> -        Map<String, V> map = new HashMap();
> +        Map<String, V> map = new HashMap<String, V>();
>          for (int i = 0; i < data.length;) {
>              map.put((String) data[i++], (V) data[i++]);
>          }
> @@ -201,7 +201,7 @@ public class UtilMisc {
>      }
>
>      public static <T> Set<T> makeSetWritable(Collection<? extends T> col)
> {
> -        Set<T> result = new HashSet<T>();
> +        Set<T> result = new LinkedHashSet<T>();
>          if (col != null) result.addAll(col);
>          return result;
>      }
> @@ -214,7 +214,7 @@ public class UtilMisc {
>       */
>      public static <V> void makeMapSerializable(Map<String, V> map) {
>          // now filter out all non-serializable values
> -        Set<String> keysToRemove = new HashSet<String>();
> +        Set<String> keysToRemove = new LinkedHashSet<String>();
>          for (Map.Entry<String, V> mapEntry: map.entrySet()) {
>              Object entryValue = mapEntry.getValue();
>              if (entryValue != null && !(entryValue instanceof
> Serializable)) {
> @@ -313,7 +313,7 @@ public class UtilMisc {
>          if (c instanceof Set<?>) {
>              theSet = (Set<T>) c;
>          } else {
> -            theSet = new HashSet<T>();
> +            theSet = new LinkedHashSet<T>();
>              c.remove(null);
>              theSet.addAll(c);
>          }
> @@ -325,7 +325,7 @@ public class UtilMisc {
>       * @return The resulting Set
>       */
>      public static <T> Set<T> toSet(T obj1) {
> -        Set<T> theSet = new HashSet<T>();
> +        Set<T> theSet = new LinkedHashSet<T>();
>          theSet.add(obj1);
>          return theSet;
>      }
> @@ -335,7 +335,7 @@ public class UtilMisc {
>       * @return The resulting Set
>       */
>      public static <T> Set<T> toSet(T obj1, T obj2) {
> -        Set<T> theSet = new HashSet<T>();
> +        Set<T> theSet = new LinkedHashSet<T>();
>          theSet.add(obj1);
>          theSet.add(obj2);
>          return theSet;
> @@ -346,7 +346,7 @@ public class UtilMisc {
>       * @return The resulting Set
>       */
>      public static <T> Set<T> toSet(T obj1, T obj2, T obj3) {
> -        Set<T> theSet = new HashSet<T>();
> +        Set<T> theSet = new LinkedHashSet<T>();
>          theSet.add(obj1);
>          theSet.add(obj2);
>          theSet.add(obj3);
> @@ -358,7 +358,7 @@ public class UtilMisc {
>       * @return The resulting Set
>       */
>      public static <T> Set<T> toSet(T obj1, T obj2, T obj3, T obj4) {
> -        Set<T> theSet = new HashSet<T>();
> +        Set<T> theSet = new LinkedHashSet<T>();
>          theSet.add(obj1);
>          theSet.add(obj2);
>          theSet.add(obj3);
> @@ -371,7 +371,7 @@ public class UtilMisc {
>       * @return The resulting Set
>       */
>      public static <T> Set<T> toSet(T obj1, T obj2, T obj3, T obj4, T
> obj5) {
> -        Set<T> theSet = new HashSet<T>();
> +        Set<T> theSet = new LinkedHashSet<T>();
>          theSet.add(obj1);
>          theSet.add(obj2);
>          theSet.add(obj3);
> @@ -385,7 +385,7 @@ public class UtilMisc {
>       * @return The resulting Set
>       */
>      public static <T> Set<T> toSet(T obj1, T obj2, T obj3, T obj4, T
> obj5, T obj6) {
> -        Set<T> theSet = new HashSet<T>();
> +        Set<T> theSet = new LinkedHashSet<T>();
>          theSet.add(obj1);
>          theSet.add(obj2);
>          theSet.add(obj3);
> @@ -396,7 +396,7 @@ public class UtilMisc {
>      }
>
>      public static <T> Set<T> toSet(T obj1, T obj2, T obj3, T obj4, T
> obj5, T obj6, T obj7, T obj8) {
> -        Set<T> theSet = new HashSet<T>();
> +        Set<T> theSet = new LinkedHashSet<T>();
>          theSet.add(obj1);
>          theSet.add(obj2);
>          theSet.add(obj3);
> @@ -413,7 +413,7 @@ public class UtilMisc {
>          if (collection instanceof Set<?>) {
>              return (Set<T>) collection;
>          } else {
> -            Set<T> theSet = new HashSet<T>();
> +            Set<T> theSet = new LinkedHashSet<T>();
>              theSet.addAll(collection);
>              return theSet;
>          }
> @@ -423,7 +423,7 @@ public class UtilMisc {
>          if (data == null) {
>              return null;
>          }
> -        Set<T> set = new HashSet<T>();
> +        Set<T> set = new LinkedHashSet<T>();
>          for (T value: data) {
>              set.add(value);
>          }
> @@ -560,7 +560,7 @@ public class UtilMisc {
>      public static <K, V> void addToSetInMap(V element, Map<K, Set<V>>
> theMap, K setKey) {
>          Set<V> theSet = UtilGenerics.checkSet(theMap.get(setKey));
>          if (theSet == null) {
> -            theSet = new HashSet<V>();
> +            theSet = new LinkedHashSet<V>();
>              theMap.put(setKey, theSet);
>          }
>          theSet.add(element);
>
>
>
... [show rest of quote]
Adrian Crum-3
Reply | Threaded
Open this post in threaded view
♦
♦
| More
Print post
Permalink
Sep 29, 2013; 4:38am

Re: svn commit: r1527212 - /ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilMisc.java

Adrian Crum-3
1903 posts
Done. Thanks for the reminder!

Adrian Crum
Sandglass Software
www.sandglass-software.com

On 9/28/2013 8:44 PM, Varun Bhansaly wrote:

> Hi Adrian,
>
> This will need to be backported to release 13.07.
> We should even consider creating few test cases that may help in such
> refactoring exercises.
>
> --
> Regards,
> Varun Bhansaly
>
>
> On Sat, Sep 28, 2013 at 11:54 PM, <[hidden email]> wrote:
>
>> Author: adrianc
>> Date: Sat Sep 28 18:24:53 2013
>> New Revision: 1527212
>>
>> URL: http://svn.apache.org/r1527212
>> Log:
>> Replaced HashSet with LinkedHashSet in UtilMisc.java to preserve original
>> ordering. Suggested by Leon on the dev mailing list.
>>
>> Modified:
>>      ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilMisc.java
>>
>> Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilMisc.java
>> URL:
>> http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilMisc.java?rev=1527212&r1=1527211&r2=1527212&view=diff
>>
>> ==============================================================================
>> --- ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilMisc.java
>> (original)
>> +++ ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilMisc.java Sat
>> Sep 28 18:24:53 2013
>> @@ -30,8 +30,8 @@ import java.util.ArrayList;
>>   import java.util.Collection;
>>   import java.util.Collections;
>>   import java.util.HashMap;
>> -import java.util.HashSet;
>>   import java.util.Iterator;
>> +import java.util.LinkedHashSet;
>>   import java.util.LinkedList;
>>   import java.util.List;
>>   import java.util.Locale;
>> @@ -159,7 +159,7 @@ public class UtilMisc {
>>               Debug.logInfo(e, module);
>>               throw e;
>>           }
>> -        Map<String, V> map = new HashMap();
>> +        Map<String, V> map = new HashMap<String, V>();
>>           for (int i = 0; i < data.length;) {
>>               map.put((String) data[i++], (V) data[i++]);
>>           }
>> @@ -201,7 +201,7 @@ public class UtilMisc {
>>       }
>>
>>       public static <T> Set<T> makeSetWritable(Collection<? extends T> col)
>> {
>> -        Set<T> result = new HashSet<T>();
>> +        Set<T> result = new LinkedHashSet<T>();
>>           if (col != null) result.addAll(col);
>>           return result;
>>       }
>> @@ -214,7 +214,7 @@ public class UtilMisc {
>>        */
>>       public static <V> void makeMapSerializable(Map<String, V> map) {
>>           // now filter out all non-serializable values
>> -        Set<String> keysToRemove = new HashSet<String>();
>> +        Set<String> keysToRemove = new LinkedHashSet<String>();
>>           for (Map.Entry<String, V> mapEntry: map.entrySet()) {
>>               Object entryValue = mapEntry.getValue();
>>               if (entryValue != null && !(entryValue instanceof
>> Serializable)) {
>> @@ -313,7 +313,7 @@ public class UtilMisc {
>>           if (c instanceof Set<?>) {
>>               theSet = (Set<T>) c;
>>           } else {
>> -            theSet = new HashSet<T>();
>> +            theSet = new LinkedHashSet<T>();
>>               c.remove(null);
>>               theSet.addAll(c);
>>           }
>> @@ -325,7 +325,7 @@ public class UtilMisc {
>>        * @return The resulting Set
>>        */
>>       public static <T> Set<T> toSet(T obj1) {
>> -        Set<T> theSet = new HashSet<T>();
>> +        Set<T> theSet = new LinkedHashSet<T>();
>>           theSet.add(obj1);
>>           return theSet;
>>       }
>> @@ -335,7 +335,7 @@ public class UtilMisc {
>>        * @return The resulting Set
>>        */
>>       public static <T> Set<T> toSet(T obj1, T obj2) {
>> -        Set<T> theSet = new HashSet<T>();
>> +        Set<T> theSet = new LinkedHashSet<T>();
>>           theSet.add(obj1);
>>           theSet.add(obj2);
>>           return theSet;
>> @@ -346,7 +346,7 @@ public class UtilMisc {
>>        * @return The resulting Set
>>        */
>>       public static <T> Set<T> toSet(T obj1, T obj2, T obj3) {
>> -        Set<T> theSet = new HashSet<T>();
>> +        Set<T> theSet = new LinkedHashSet<T>();
>>           theSet.add(obj1);
>>           theSet.add(obj2);
>>           theSet.add(obj3);
>> @@ -358,7 +358,7 @@ public class UtilMisc {
>>        * @return The resulting Set
>>        */
>>       public static <T> Set<T> toSet(T obj1, T obj2, T obj3, T obj4) {
>> -        Set<T> theSet = new HashSet<T>();
>> +        Set<T> theSet = new LinkedHashSet<T>();
>>           theSet.add(obj1);
>>           theSet.add(obj2);
>>           theSet.add(obj3);
>> @@ -371,7 +371,7 @@ public class UtilMisc {
>>        * @return The resulting Set
>>        */
>>       public static <T> Set<T> toSet(T obj1, T obj2, T obj3, T obj4, T
>> obj5) {
>> -        Set<T> theSet = new HashSet<T>();
>> +        Set<T> theSet = new LinkedHashSet<T>();
>>           theSet.add(obj1);
>>           theSet.add(obj2);
>>           theSet.add(obj3);
>> @@ -385,7 +385,7 @@ public class UtilMisc {
>>        * @return The resulting Set
>>        */
>>       public static <T> Set<T> toSet(T obj1, T obj2, T obj3, T obj4, T
>> obj5, T obj6) {
>> -        Set<T> theSet = new HashSet<T>();
>> +        Set<T> theSet = new LinkedHashSet<T>();
>>           theSet.add(obj1);
>>           theSet.add(obj2);
>>           theSet.add(obj3);
>> @@ -396,7 +396,7 @@ public class UtilMisc {
>>       }
>>
>>       public static <T> Set<T> toSet(T obj1, T obj2, T obj3, T obj4, T
>> obj5, T obj6, T obj7, T obj8) {
>> -        Set<T> theSet = new HashSet<T>();
>> +        Set<T> theSet = new LinkedHashSet<T>();
>>           theSet.add(obj1);
>>           theSet.add(obj2);
>>           theSet.add(obj3);
>> @@ -413,7 +413,7 @@ public class UtilMisc {
>>           if (collection instanceof Set<?>) {
>>               return (Set<T>) collection;
>>           } else {
>> -            Set<T> theSet = new HashSet<T>();
>> +            Set<T> theSet = new LinkedHashSet<T>();
>>               theSet.addAll(collection);
>>               return theSet;
>>           }
>> @@ -423,7 +423,7 @@ public class UtilMisc {
>>           if (data == null) {
>>               return null;
>>           }
>> -        Set<T> set = new HashSet<T>();
>> +        Set<T> set = new LinkedHashSet<T>();
>>           for (T value: data) {
>>               set.add(value);
>>           }
>> @@ -560,7 +560,7 @@ public class UtilMisc {
>>       public static <K, V> void addToSetInMap(V element, Map<K, Set<V>>
>> theMap, K setKey) {
>>           Set<V> theSet = UtilGenerics.checkSet(theMap.get(setKey));
>>           if (theSet == null) {
>> -            theSet = new HashSet<V>();
>> +            theSet = new LinkedHashSet<V>();
>>               theMap.put(setKey, theSet);
>>           }
>>           theSet.add(element);
>>
>>
>>
>
... [show rest of quote]
Free forum by Nabble Edit this page