commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From robert burrell donkin <>
Subject Re: [validator] Refactoring?
Date Wed, 10 Jul 2002 18:34:23 GMT
hi joe

there don't seem to be any validator developers around at the moment so i'
ll do my best.

the most important thing is to make contact with the developers. so you've 
done the right thing but you might need to hang around for a while until 
it's sorted.

i also think that several small, focused, well explained patches are more 
likely to be applied than one big one. that way, they can be checked and 
applied one by one without having to go through loads of code.

- robert

On Thursday, July 4, 2002, at 03:34 PM, Joe Germuska wrote:

> Hello:
> I was interested in understanding Validator a little better, and it seems 
> like for now the best way to do that is in the source code.
> As I'm looking at it, I'm applying a few refactorings, mostly just to 
> clarify the flow (Validator.validate() is over 200 lines long!)  Is it 
> enough to just make sure that the tests all continue to pass and then 
> submit a patch later?
> The net number of lines changed will probably be pretty many, so I 
> thought it might be smarter to coordinate with Dave W and any other 
> people who are particularly interested in the validator component, 
> especially because I haven't used it extensively.  Perhaps it makes sense 
> to do more and smaller patches, if not in CVS then privately, rather than 
> have one huge patch that never gets committed because it's hard to 
> understand.
> I don't claim to understand Validator well enough to do any refactorings 
> that go across classes -- I'm just extracting small private methods where 
> I think it helps clarify things.
> Joe
> --
> To unsubscribe, e-mail:   <mailto:commons-dev-unsubscribe@jakarta.apache.
> org>
> For additional commands, e-mail: <mailto:commons-dev-help@jakarta.apache.
> org>

To unsubscribe, e-mail:   <>
For additional commands, e-mail: <>

View raw message