[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-kimageshop
Subject:    Re: Animation paintop, bad performance
From:       Dmitry Kazakov <dimula73 () gmail ! com>
Date:       2010-10-10 8:57:32
Message-ID: AANLkTi=cUiKBezteCQxagqNsT76NTTJvYcC6356KScmO () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


>
>  use the COMPOSITE_COPY composite op.
>
> Reason #1 Avoid duplicating existing functionnality :) Especially since
> there might be optimization
>
> Reason #2 In your case, there will be a limited amount of memcpy, and
> memory usuage, as tiles will be shared between src and dst
>
> And in any case, avoid using deprecated iterators in new code.
>
>
Well, the problem here is that KisPainter::bitBlt doe not allow to read from
oldRawData. I think the interface should be modified somehow.

-- 
Dmitry Kazakov

[Attachment #5 (text/html)]

<div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt \
0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div \
style="font-family: &#39;DejaVu Sans Mono&#39;; font-size: 9pt; font-weight: 400; \
font-style: normal;">

<p style="margin: 0px; text-indent: 0px;"> </p>
<p style="margin: 0px; text-indent: 0px;">use the COMPOSITE_COPY composite op.</p>
<p style="margin: 0px; text-indent: 0px;"></p>
<p style="margin: 0px; text-indent: 0px;">Reason #1 Avoid duplicating existing \
functionnality :) Especially since there might be optimization</p> <p style="margin: \
0px; text-indent: 0px;">Reason #2 In your case, there will be a limited amount of \
memcpy, and memory usuage, as tiles will be shared between src and dst</p> <p \
style="margin: 0px; text-indent: 0px;"></p> <p style="margin: 0px; text-indent: \
0px;">And in any case, avoid using deprecated iterators in new code.</p> <p \
style="margin: 0px; text-indent: 0px;"></p></div></blockquote></div><br>Well, the \
problem here is that KisPainter::bitBlt doe not allow to read from oldRawData. I \
                think the interface should be modified somehow.<br><br>
-- <br>Dmitry Kazakov<br>



_______________________________________________
kimageshop mailing list
kimageshop@kde.org
https://mail.kde.org/mailman/listinfo/kimageshop


[prev in list] [next in list] [prev in thread] [next in thread]