Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-10-04 Thread Jörg Schaible
Jörg Schaible wrote: > Gary Gregory wrote: > >> On Wed, Oct 3, 2012 at 7:28 AM, Jörg Schaible >> wrote: >> >>> Jörg Schaible wrote: >>> >>> > >>> > sebb wrote: >>> >>> [snip] >>> >>> >> I find the naming convention rather difficult to follow. >>> >> >>> >> For example, the letter 's' sometimes

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-10-03 Thread Gary Gregory
I like the enum idea too. G On Wed, Oct 3, 2012 at 2:12 PM, Duncan Jones wrote: > On 3 October 2012 18:24, Jörg Schaible wrote: > > Matt Benson wrote: > > > >> Urgh; I find these method names rather painful. Why wouldn't we > >> simply provide endianness and bit ordering as enums, and paramet

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-10-03 Thread Duncan Jones
On 3 October 2012 18:24, Jörg Schaible wrote: > Matt Benson wrote: > >> Urgh; I find these method names rather painful. Why wouldn't we >> simply provide endianness and bit ordering as enums, and parameterize >> accordingly? > > Because the algorithm is different (although similar) every time and

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-10-03 Thread Jörg Schaible
Matt Benson wrote: > Urgh; I find these method names rather painful. Why wouldn't we > simply provide endianness and bit ordering as enums, and parameterize > accordingly? Because the algorithm is different (although similar) every time and not all combinations are implemented? Honestly, we wo

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-10-03 Thread Jörg Schaible
Gary Gregory wrote: > On Wed, Oct 3, 2012 at 7:28 AM, Jörg Schaible > wrote: > >> Jörg Schaible wrote: >> >> > >> > sebb wrote: >> >> [snip] >> >> >> I find the naming convention rather difficult to follow. >> >> >> >> For example, the letter 's' sometimes means 'array' and sometimes >> >> means

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-10-03 Thread Gary Gregory
Also note that we have org.apache.commons.*io*.EndianUtils which does not do the same thing but it a good class name. Gary On Wed, Oct 3, 2012 at 10:51 AM, James Carman wrote: > Agreed. This is very cryptic, IMHO. > > On Wed, Oct 3, 2012 at 10:42 AM, Matt Benson wrote: > > Urgh; I find these m

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-10-03 Thread James Carman
Agreed. This is very cryptic, IMHO. On Wed, Oct 3, 2012 at 10:42 AM, Matt Benson wrote: > Urgh; I find these method names rather painful. Why wouldn't we > simply provide endianness and bit ordering as enums, and parameterize > accordingly? > > Matt > > On Wed, Oct 3, 2012 at 9:07 AM, Gary Greg

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-10-03 Thread Matt Benson
Urgh; I find these method names rather painful. Why wouldn't we simply provide endianness and bit ordering as enums, and parameterize accordingly? Matt On Wed, Oct 3, 2012 at 9:07 AM, Gary Gregory wrote: > On Wed, Oct 3, 2012 at 7:28 AM, Jörg Schaible wrote: > >> Jörg Schaible wrote: >> >> > >

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-10-03 Thread Gary Gregory
On Wed, Oct 3, 2012 at 7:28 AM, Jörg Schaible wrote: > Jörg Schaible wrote: > > > > > sebb wrote: > > [snip] > > >> I find the naming convention rather difficult to follow. > >> > >> For example, the letter 's' sometimes means 'array' and sometimes means > >> 'string' Also M0 is not obvious as an

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-10-03 Thread Jörg Schaible
Jörg Schaible wrote: > > sebb wrote: [snip] >> I find the naming convention rather difficult to follow. >> >> For example, the letter 's' sometimes means 'array' and sometimes means >> 'string' Also M0 is not obvious as an abbreviation, nor is BeM0 > > I am all open for better names, all *I*

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-09-29 Thread Jörg Schaible
sebb wrote: > On 28 September 2012 18:47, Jörg Schaible wrote: >> sebb wrote: >> [snip] >>> Unless there is a significant improvement across several Java >>> versions, I'm -1 on the change as the code is now more obscure. >> >> Done. > > Thanks! > >> Any comment on replacing the standard hex

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-09-29 Thread Gary Gregory
On Sep 29, 2012, at 6:58, "Jörg Schaible" wrote: > Gary Gregory wrote: > >> On Fri, Sep 28, 2012 at 1:47 PM, Jörg Schaible >> wrote: >> >>> sebb wrote: > > [snip] > Unless there is a significant improvement across several Java versions, I'm -1 on the change as the code is now more obscu

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-09-29 Thread Jörg Schaible
Gary Gregory wrote: > On Fri, Sep 28, 2012 at 1:47 PM, Jörg Schaible > wrote: > >> sebb wrote: [snip] >> > Unless there is a significant improvement across several Java >> > versions, I'm -1 on the change as the code is now more obscure. >> >> Done. >> >> Any comment on replacing the standard h

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-09-28 Thread sebb
On 28 September 2012 18:47, Jörg Schaible wrote: > sebb wrote: > >> On 28 September 2012 08:17, Jörg Schaible >> wrote: >>> Hi Sebb, >>> >>> sebb wrote: >>> On 27 September 2012 23:53, wrote: > Author: joehni > Date: Thu Sep 27 22:53:46 2012 > New Revision: 1391258 > >

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-09-28 Thread Gary Gregory
On Fri, Sep 28, 2012 at 1:47 PM, Jörg Schaible wrote: > sebb wrote: > > > On 28 September 2012 08:17, Jörg Schaible > > wrote: > >> Hi Sebb, > >> > >> sebb wrote: > >> > >>> On 27 September 2012 23:53, wrote: > Author: joehni > Date: Thu Sep 27 22:53:46 2012 > New Revision: 13912

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-09-28 Thread Jörg Schaible
sebb wrote: > On 28 September 2012 08:17, Jörg Schaible > wrote: >> Hi Sebb, >> >> sebb wrote: >> >>> On 27 September 2012 23:53, wrote: Author: joehni Date: Thu Sep 27 22:53:46 2012 New Revision: 1391258 URL: http://svn.apache.org/viewvc?rev=1391258&view=rev Log:

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-09-28 Thread sebb
On 28 September 2012 08:17, Jörg Schaible wrote: > Hi Sebb, > > sebb wrote: > >> On 27 September 2012 23:53, wrote: >>> Author: joehni >>> Date: Thu Sep 27 22:53:46 2012 >>> New Revision: 1391258 >>> >>> URL: http://svn.apache.org/viewvc?rev=1391258&view=rev >>> Log: >>> Use conversion tables fo

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-09-28 Thread Benedikt Ritter
2012/9/28 Jörg Schaible : > Hi Sebb, > > sebb wrote: > >> On 27 September 2012 23:53, wrote: >>> Author: joehni >>> Date: Thu Sep 27 22:53:46 2012 >>> New Revision: 1391258 >>> >>> URL: http://svn.apache.org/viewvc?rev=1391258&view=rev >>> Log: >>> Use conversion tables for boolean arrays. >> >>

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-09-28 Thread Jörg Schaible
Hi Sebb, sebb wrote: > On 27 September 2012 23:53, wrote: >> Author: joehni >> Date: Thu Sep 27 22:53:46 2012 >> New Revision: 1391258 >> >> URL: http://svn.apache.org/viewvc?rev=1391258&view=rev >> Log: >> Use conversion tables for boolean arrays. > > I think the previous code was much cleare

Re: svn commit: r1391258 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/Conversion.java

2012-09-27 Thread sebb
On 27 September 2012 23:53, wrote: > Author: joehni > Date: Thu Sep 27 22:53:46 2012 > New Revision: 1391258 > > URL: http://svn.apache.org/viewvc?rev=1391258&view=rev > Log: > Use conversion tables for boolean arrays. I think the previous code was much clearer. At first sight it now looks as t