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: [email protected] To: [email protected] 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: [email protected] > For additional commands, e-mail: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
