Adrian Crum wrote:
> Adam,
> 
> Thanks for the tips!
> 
> The public modifier and @Deprecated in the Delegator interface are
> carried over from GenericDelegator.java. I used Eclipse to extract the
> interface - hence the public modifier. David Jones deprecated the
> methods - not me. We can remove the deprecated methods when they are no
> longer used.

Nono.

You've added a *new* interface.  Adding immediately deprecated methods
to a *new* interface or class is stupid.

I'm not suggesting removing the methods from GenericDelegator.  Just
the definitions from the interface.

Reply via email to