Re: [CODEC] Hex class confusing mix of static and instance methods

2008-07-14 Thread James Ring
Hey Sebb, On Mon, Jul 14, 2008 at 5:08 AM, sebb <[EMAIL PROTECTED]> wrote: > The Hex class has a mix of static and instance methods, which is a bit > confusing. > > The class does not have any state at all, so all the methods could be static. > > Perhaps we should consider adding static versions o

[continuum] BUILD SUCCESSFUL: Commons - Commons Pipeline (Sandbox) - Build Def:

2008-07-14 Thread [EMAIL PROTECTED]
Online report : http://vmbuild.apache.org/continuum/buildResult.action?buildId=103163&projectId=187 Build statistics: State: Ok Previous Build: No previous build. Started at: Mon 14 Jul 2008 16:36:36 -0700 Finished at: Mon 14 Jul 2008 16:37:04 -0700 Total time: 28s Build Trigger: Forced B

[continuum] BUILD SUCCESSFUL: Commons - Commons Pipeline (Sandbox) - Build Def:

2008-07-14 Thread [EMAIL PROTECTED]
Online report : http://vmbuild.apache.org/continuum/buildResult.action?buildId=103162&projectId=187 Build statistics: State: Ok Previous State: Failed Started at: Mon 14 Jul 2008 16:35:48 -0700 Finished at: Mon 14 Jul 2008 16:36:26 -0700 Total time: 37s Build Trigger: Forced Build Number:

[continuum] BUILD SUCCESSFUL: Commons - Commons Discovery - Build Def:

2008-07-14 Thread [EMAIL PROTECTED]
Online report : http://vmbuild.apache.org/continuum/buildResult.action?buildId=103161&projectId=362 Build statistics: State: Ok Previous State: Failed Started at: Mon 14 Jul 2008 15:45:27 -0700 Finished at: Mon 14 Jul 2008 15:45:49 -0700 Total time: 22s Build Trigger: Forced Build Number:

Re: [CODEC] Base64 decode() handling of invalid encoded data

2008-07-14 Thread sebb
On 14/07/2008, Jochen Wiedmann <[EMAIL PROTECTED]> wrote: > On Mon, Jul 14, 2008 at 5:21 PM, sebb <[EMAIL PROTECTED]> wrote: > > > Just noticed https://issues.apache.org/jira/browse/CODEC-68 which > > implies that at least one person is expecting to be able to include > > invalid base64 characte

Re: [VOTE] Release version 1.1 of commons-build-plugin based on RC1

2008-07-14 Thread Jörg Schaible
+1 Builds fine from source zip on JDK 1.4 and 1.5. - Jörg Niall Pemberton wrote: > I'd like to release version 1.1 commons-build-plugin. > > The main change is upgrading to the latest Ant 1.7.1 release which > fixes a couple of problems (thanks to Dennis!) > > Release Notes: > http://people.a

Re: [CODEC] Base64 decode() handling of invalid encoded data

2008-07-14 Thread Jochen Wiedmann
On Mon, Jul 14, 2008 at 5:21 PM, sebb <[EMAIL PROTECTED]> wrote: > Just noticed https://issues.apache.org/jira/browse/CODEC-68 which > implies that at least one person is expecting to be able to include > invalid base64 characters without a problem - in that case, after the > PAD characters. > > S

Re: [CODEC] Base64 decode() handling of invalid encoded data

2008-07-14 Thread sebb
On 14/07/2008, Jochen Wiedmann <[EMAIL PROTECTED]> wrote: > On Mon, Jul 14, 2008 at 3:37 PM, sebb <[EMAIL PROTECTED]> wrote: > > > Should have tried it before posting ... > > > > Both IOException and DecoderException are checked, and so this means > > that the method signature for > > > > pub

Re: [CODEC] Base64 decode() handling of invalid encoded data

2008-07-14 Thread Jochen Wiedmann
On Mon, Jul 14, 2008 at 3:37 PM, sebb <[EMAIL PROTECTED]> wrote: > Should have tried it before posting ... > > Both IOException and DecoderException are checked, and so this means > that the method signature for > > public static byte[] decodeBase64() > > would need to change - or the method needs

Re: [CODEC] Base64 decode() handling of invalid encoded data

2008-07-14 Thread sebb
On 14/07/2008, Jochen Wiedmann <[EMAIL PROTECTED]> wrote: > On Mon, Jul 14, 2008 at 2:24 PM, sebb <[EMAIL PROTECTED]> wrote: > > Just looked at this again. > > > > Using IOException causes problems, because the interface BinaryDecoder > > does not declare that as a possible exception. > > > >

Re: [CODEC] Base64 decode() handling of invalid encoded data

2008-07-14 Thread Jochen Wiedmann
On Mon, Jul 14, 2008 at 2:24 PM, sebb <[EMAIL PROTECTED]> wrote: > Just looked at this again. > > Using IOException causes problems, because the interface BinaryDecoder > does not declare that as a possible exception. > > How about using DecoderException instead? > > Seems just as appropriate to me

Re: [CODEC] Base64 decode() handling of invalid encoded data

2008-07-14 Thread sebb
Just looked at this again. Using IOException causes problems, because the interface BinaryDecoder does not declare that as a possible exception. How about using DecoderException instead? Seems just as appropriate to me. On 25/06/2008, Julius Davies <[EMAIL PROTECTED]> wrote: > +1 for IOExceptio

[CODEC] Hex class confusing mix of static and instance methods

2008-07-14 Thread sebb
The Hex class has a mix of static and instance methods, which is a bit confusing. The class does not have any state at all, so all the methods could be static. Perhaps we should consider adding static versions of the instance methods (and have the instance methods call these), with a view to perh

Re: [cli] CLI 2 State of Play

2008-07-14 Thread Dennis Lundberg
Tobias Bocanegra wrote: On 7/13/08, Oliver Heger <[EMAIL PROTECTED]> wrote: Henri Yandell schrieb: On Sat, Jul 12, 2008 at 9:12 AM, Russel Winder <[EMAIL PROTECTED]> wrote: On Sat, 2008-07-12 at 16:02 +0100, Niall Pemberton wrote: I saw from the following blog[1] that Hadoop is using CLI 2

Re: [VOTE] Release version 1.1 of commons-build-plugin based on RC1

2008-07-14 Thread Luc Maisonobe
Niall Pemberton a écrit : > I'd like to release version 1.1 commons-build-plugin. > > The main change is upgrading to the latest Ant 1.7.1 release which > fixes a couple of problems (thanks to Dennis!) > > Release Notes: > http://people.apache.org/~niallp/cbp-1.1-rc1/RELEASE-NOTES.txt > > Artifa

Re: [cli] CLI 2 State of Play

2008-07-14 Thread Tobias Bocanegra
On 7/13/08, Oliver Heger <[EMAIL PROTECTED]> wrote: > Henri Yandell schrieb: > > > On Sat, Jul 12, 2008 at 9:12 AM, Russel Winder > > <[EMAIL PROTECTED]> wrote: > > > > > On Sat, 2008-07-12 at 16:02 +0100, Niall Pemberton wrote: > > > > > > > I saw from the following blog[1] that Hadoop is using CL