KisImage::undo() - what is it?

[Dmitry Kazakov]

And one more question: why in different places we add commands to different
places, like

krita/ui/kis_selection_manager.cc:
m_view->document()->addCommand(t);
...
image->undoAdapter()->addCommand(t);

kis_mask_manager.cc:
m_view->undoAdapter()->addCommand(t)


It looks quite weird, especially the first line. What if leave only one way
to add it?


On Sat, Jun 5, 2010 at 12:59 PM, Dmitry Kazakov <dimula73 at gmail.com> wrote:

> Well, sorry, most of them are consistent, but i still don't have a clue
> what it is for...
>
>
> On Sat, Jun 5, 2010 at 12:57 PM, Dmitry Kazakov <dimula73 at gmail.com>wrote:
>
>> Hi, All!
>>
>> I still can't understand what KisImage::undo() method is supposed to do?
>> It seems like it reports whether it is needed to enable/disable undo
>> capabilities of the image or something like that. What is it used for? Is it
>> still actual, or it is just a historical artefact? There is a plenty of code
>> around, that checks this state of the image, but many instances of it look
>> like they were copy-pasted form somewhere, because they are not consistent.
>> What should i do with them?
>>
>> --
>> Dmitry Kazakov
>>
>
>
>
> --
> Dmitry Kazakov
>



-- 
Dmitry Kazakov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kimageshop/attachments/20100605/daa221ae/attachment.htm