I understand what you're suggesting. But if the intention is to replace

GenericDelegator delegator = GenericDelegator.getDelegator(delegatorName);

with

Delegator delegator = (Delegator) UtilObject.getObjectFromFactory(DelegatorFactory.class, delegatorName);

then the Delegator interface will need to support the same methods as GenericDelegator.

-Adrian

Adam Heath wrote:
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