Bug 234038 - Duplicate layer fills memory
Summary: Duplicate layer fills memory
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: General (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR major
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-04-11 13:45 UTC by SirPigeonz
Modified: 2010-08-20 15:08 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description SirPigeonz 2010-04-11 13:45:59 UTC
Version:           trunk (using KDE 4.4.2)
OS:                Linux
Installed from:    Unlisted Binary Package

I had 2000x2500 image, on single layer, when I pressed "duplicate layer or mask", it filled my memory.
Comment 1 Cyrille Berger 2010-04-11 14:02:25 UTC
Yes currently, the full history of a paint layer is duplicated with the layer (and if the newly created layer is never edited, it is not purged).
Comment 2 Halla Rempt 2010-04-11 17:02:59 UTC
Ew. That cannot serve any useful purpose, can it?
Comment 3 Cyrille Berger 2010-04-11 19:11:15 UTC
I don't know, I wanted to ask Dmitry about that.
Comment 4 Dmitry Kazakov 2010-04-12 16:23:00 UTC
Well, i haven't tested it yet, but i guess this is the pooler preallocating tiles for future use. This is quite logical, especially when you are going to edit newly created (or source) layer. It is the common case, i guess. 
Though it could be quite cool feature for preallocated tiles to decay with the time.
Comment 5 Dmitry Kazakov 2010-04-12 16:24:26 UTC
And i have to check about "the full history"...
Comment 6 Halla Rempt 2010-08-11 22:34:02 UTC
BUGDAY
Comment 7 Dmitry Kazakov 2010-08-20 15:08:57 UTC
SVN commit 1165938 by dkazakov:

Do not save the history of the cloned data manager.

After a small discussion we decided there is no usecase for it.
BUG:234038


 M  +4 -2      kis_tiled_data_manager.cc  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1165938