Painting on a temporary layer

[Boudewijn Rempt]

On Sunday 26 September 2004 16:59, Bart Coppens wrote:

> I thought about it, but I think there are too many objections to this
> approach: it doesn't really belong in KisImage, I'd probably have to remove
> it again later if I can use the undo as source, etc. Also, we'd be working
> around the problem instead of fixing it, because I think the current
> behaviour is wrong.

It's not something I would have problems with -- I'm a very iterative coder. 
Add a little bit, see that it works, add some more, replace the previous bit. 
I've been a sculptor, and I code like that. Add some wax or clay, scrape it 
away. It's all very plastic. (Note... I've had a very full meal, so I'm 
feeling philosophical :-).

> That's indeed a good idea, but after looking at kis_tile_command, I think
> this could get more difficult than I thought. The undo information is
> efficiently stored in an std::map object, so getting it back to use as a
> source for painting or making previews might be a challenge...

I think it'll pay to start thinking about meeting that challenge. It's another 
requirement for the pixel manager API we've discussed some time ago. (Casper, 
are you listening? Any progress?).

>
>
> As you might have seen, I committed an updated version of
> kis_tool_freehand.cc This version implements the 'fix' I mentioned in my
> last mail. To test out this code, you only have to uncomment the
> setUseTempLayer( true );

As far as I'm concerned, you can enable this by default. Slow and jumpy is 
better than not working correctly. If something works a little bit it's an 
incitement to others who feel irritated to fix it.

> lines in kis_tool_filter.cc and in kis_tool_duplicate.cc The only thing
> that doesn't quite work the way I expected is the duplicate tool: it
> appears a bit 'jumpy', but I have no idea why.

I'll experiment with it a bit, but I suspect some basic slowness.
-- 
Boudewijn Rempt | "Geef mij maar zuurtjes."
http://www.valdyas.org/fading/index.cgi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/kimageshop/attachments/20040926/046f9eda/attachment.pgp