<br><br><div class="gmail_quote">On Sat, Sep 26, 2009 at 10:44 AM, Boudewijn Rempt <span dir="ltr"><<a href="mailto:boud@valdyas.org">boud@valdyas.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On Wednesday 23 September 2009, Dmitry Kazakov wrote:<br>
<br>
> This is a singlepatch version. It can be easily applied to trunk without<br>
> conflicts. It's good for testing, not for reading:<br>
> <a href="http://dimula73.narod.ru/01_layers_masks_refactoring_singlepatch.diff" target="_blank">http://dimula73.narod.ru/01_layers_masks_refactoring_singlepatch.diff</a><br></div></blockquote><div><br>Yes, recent commits broke it. I'm compiling newly merged one.<br>
<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im">
</div>There are a couple of stylistic issues I'm not too happy with, like<br>
rearranging the order of methods in a header and some renaming.</blockquote><div>I just wanted to make them more readable by grouping resembling methods.<br><br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Right now, for<br>
someone who's been working with Krita for a longer time, confusion is<br>
guaranteed between the term 'projection; and the new term 'cache'.<br></blockquote><div><br>Btw, there is no new term 'cache' =) Do you mean KisGroupLayer::resetCache()?<br>I changed it from resetProjection to resetCache because KisAdjustmentLayer had already had similar method. They both should have the same name. And their name can't be resetProjection because now they reset 'original' paint device instead of 'projection'<br>
<br>I'll write about projections, originals and painDevices in a reply to your next mail.<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
These big refactors go better if we keep the renaming to the end, I think. I'm<br>
still digesting your other mail, it's as long as the patch!<br></blockquote><div><br>=) <br></div></div><br clear="all"><br>-- <br>Dmitry Kazakov<br>