Clean-up for KisLayerManager and KisMaskManager

Boudewijn Rempt boud at valdyas.org
Sat Apr 17 21:20:46 CEST 2010


On Friday 16 April 2010, Dmitry Kazakov wrote:
> Hi, All!
> 
> Here is a patch that clean-ups/fixes some flaws in layer creation part. I
> don't know where to commit it actually: trunk or koffice-ko...
> 
> What it does:
> 
>  o Fixes a memory leak in KisMaskManager when the creation of the mask
>     is canceled
>  o Removed superfluous setDirty() from both managers those caused double
>     updates of the image when creating the layer
>  o Moved activateNode functionality to KisNodeManager

All this sounds like trunk...


>  o Removed deprecated default functions for creating layers and
>     masks. Now the parent of the new mask is chosen by
>     KisNodeManager::getNewNodeLocation.


But this sounds like the branch.

In the end, the memory leak is the important thing, so if you cannot split it, 
it should go into trunk, I think.

-- 
Boudewijn Rempt | http://www.valdyas.org


More information about the kimageshop mailing list