And one more question: why in different places we add commands to different places, like<br><br>krita/ui/kis_selection_manager.cc:<br>m_view->document()->addCommand(t);<br>...<br>image->undoAdapter()->addCommand(t);<br>
<br>kis_mask_manager.cc:<br>m_view->undoAdapter()->addCommand(t)<br><br><br>It looks quite weird, especially the first line. What if leave only one way to add it?<br><br><br><div class="gmail_quote">On Sat, Jun 5, 2010 at 12:59 PM, Dmitry Kazakov <span dir="ltr"><<a href="mailto:dimula73@gmail.com">dimula73@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">Well, sorry, most of them are consistent, but i still don't have a clue what it is for...<div>
<div></div><div class="h5"><br><br><div class="gmail_quote">On Sat, Jun 5, 2010 at 12:57 PM, Dmitry Kazakov <span dir="ltr"><<a href="mailto:dimula73@gmail.com" target="_blank">dimula73@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">Hi, All!<br><br>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?<br clear="all">
<br>-- <br><font color="#888888">Dmitry Kazakov<br>
</font></blockquote></div><br><br clear="all"><br></div></div>-- <br><font color="#888888">Dmitry Kazakov<br>
</font></blockquote></div><br><br clear="all"><br>-- <br>Dmitry Kazakov<br>