Re: [beanutils2] SANDBOX-472

2014-08-17 Thread Benedikt Ritter
Hello Yogesh, I'll have a look at your patch later this week! Thanks, Benedikt 2014-08-14 6:24 GMT+02:00 Yogesh Rao : > Hi Benedikt, > > Have attached the updated patch in JIRA. Please do apply it if everything > is found okay. > > Regards, > -Yogesh > > > On Wed, Jul 23, 2014 at 11:18 AM, Ben

Re: Fwd: svn commit: r1617904 - /commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/compressors/xz/XZUtils.java

2014-08-17 Thread sebb
On 17 August 2014 13:33, Gary Gregory wrote: > Seems to me we should use an enumerated instead of related constants. > > Any reason why not? At present the code uses an AtomicInteger to hold the setting. This would need to be changed. I suspect just using a volatile enum would be sufficient as e

Fwd: svn commit: r1617904 - /commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/compressors/xz/XZUtils.java

2014-08-17 Thread Gary Gregory
Seems to me we should use an enumerated instead of related constants. Any reason why not? Gary Original message From: bode...@apache.org Date:08/14/2014 05:46 (GMT-05:00) To: comm...@commons.apache.org Subject: svn commit: r1617904 - /commons/proper/compress/trunk/src/mai