Hi Adrian,

I'm not happy with the change of the Graphics2DImagePainter interface in
XGCommons that you seem to have done locally. This interface is targeted
at painting only. IMO it should not assume that every image also has a
URI associated with it for whatever that URI will then be used. For
example, foreign objects don't have an explicit URI. If you need the
MIME type and the original URI of an image inside the
AFPGraphics2DAdapter, you can pass those in through the RendererContext.
For example, AFPRenderer.drawImage() already has the ImageInfo object
and would simply need to put it in the RendererContext. Extending the
Graphics2DPainter interface just for transporting some
non-painting-related information that only the AFP renderer needs is not
justification enough for this change if you ask me. Plus this breaks
other applications that rely on this interface.

On 02.06.2008 16:30:40 acumiskey wrote:
> Author: acumiskey
> Date: Mon Jun  2 07:30:39 2008
> New Revision: 662448
> URL: http://svn.apache.org/viewvc?rev=662448&view=rev
> Log:
> * getImageUri() implementation provided on Graphics2DAdaptor implementations.
> * Refactored data object creation abilities from 
> AbstractResourceGroupContainer into a new DataObjectFactory class. 
> * Added mime type to ObjectType inner class in Registry for warn/error 
> logging purposes.
> * Fixed problem with handling ObjectContainer objects in IncludeObject.
> * Added documentation for configuring AFP resource group files.

Jeremias Maerki

Reply via email to