Jonathon -- Improov wrote:
Seems a bit counter-intuitive, that design pattern. Or we could rename ApacheFopWorker to ApacheFopFactoryHelper.
You're right, but my thinking is the ApacheFopFactory class could be phased out. From my perspective, it seems kind of silly to have a class that does nothing more than return an instance of another class.
-Adrian
