commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Simone Tripodi <simonetrip...@apache.org>
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:26:58 GMT
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


Mime
View raw message