> -----Original Message----- > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] > Sent: 22 June 2003 04:51 > To: [EMAIL PROTECTED] > Subject: cvs commit: jakarta- > cactus/framework/src/java/share/org/apache/cactus > AbstractWebServerTestCase.java
[snip] > - if (getWrappedTest() != this) > + if (wrappingATest()) The refactoring is good but I don't much like the wrappingATest() method name... What about hasWrappedTest() or something like this (I like to prefix methods that return a boolean with "is" or "has")? Thanks -Vincent --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
