koffice/krita

[Adrian Page]

On Friday 27 August 2004 5:35 pm, Boudewijn Rempt wrote:
> 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...

I'll see if I can come up with a patch.

Adrian