koffice/krita

[Boudewijn Rempt]

On Fri, 27 Aug 2004, Adrian Page wrote:

> On Friday 27 August 2004 2:00 pm, Boudewijn Rempt wrote:
> > 	/**
> > 	 * Render the contents of the projection image onto painter.
> > 	 */
> > 	virtual void render(KisImageSP projection, QPainter& painter, Q_INT32 x,
> > Q_INT32 y, Q_INT32 width, Q_INT32 height) = 0;
> >
> > 	/**
> > 	 * Create a QImage from the specified Krita image.
> > 	 */
> > 	virtual QImage convertToImage(KisImageSP image, Q_INT32 x, Q_INT32 y,
> > Q_INT32 width, Q_INT32 height) const = 0;
> >
> > 	/**
> > 	 * Create a QImage from the data in the specified tile manager.
> > 	 */
> > 	virtual QImage convertToImage(KisTileMgrSP tm, Q_UINT32 depth, Q_INT32 x,
> > Q_INT32 y, Q_INT32 width, Q_INT32 height) const = 0;
>
> I was wondering whether these functions should belong to the colour strategy
> as they involve quite a bit more than its main job of interpreting raw image
> data.
>
>
Good point -- they're there because you need the image data interpreted to
be able to create QImage for display, but if you've got a suggestion for
an alternative, I'm all ears. If we can have something generic that does the complex
bits...