commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From b...@systemoutprintln.de
Subject Re: svn commit: r1238968 - in /commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2: BeanUtils.java ClassLoaderBuilder.java DefaultClassLoaderBuilder.java
Date Wed, 01 Feb 2012 08:51:11 GMT
You got me again. Sorry about that. I thought, this time I could make a good suggestion. Damn
it ;-) I'm still having trouble reading the svn commit mails. Next time, I'll better check
the commit from my IDE.

----- Original Message -----
From: simonetripodi@apache.org
To: dev@commons.apache.org
Date: 01.02.2012 09:26:58
Subject: Re: svn commit: r1238968 - in /commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2:
BeanUtils.java ClassLoaderBuilder.java DefaultClassLoaderBuilder.java


> Guten morgen,
> 
>> I don't think that "onClassName" is a good name for that method, because fluent invocations
will look like:
>> onClassName("java.lang.Object").invokeStaticMethod(...)...
>> We do not invoke a method on a class name, but on a class. I think
>> onClass(String className) is self-explanatory enough and it would be similar to invokeStaticMethod(String
methodName).
>> 
>> What do you think?
> 
> I think you didn't checkout the code and/or didn't have a look at how
> methods are chained in the patch :)
> 
> onClassName( "com.acme.Something" ).loadWithClassLoader( myClassLoader
> ).invokeStaticMethod(...)
> 
> 
> ˆˆˆˆˆˆˆˆˆˆhave a look at alternatives
> 
> best,
> -Simo
> 
> http://people.apache.org/~simonetripodi/
> http://simonetripodi.livejournal.com/
> http://twitter.com/simonetripodi
> http://www.99soft.org/
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org






---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message