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

List:       kde-kimageshop
Subject:    Re: Krita useable for Blender movies
From:       Dmitry Kazakov <dimula73 () gmail ! com>
Date:       2009-10-23 15:37:39
Message-ID: ae32c1ef0910230837v4432f804m3ddd4132c21ddedc () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


On Fri, Oct 23, 2009 at 4:52 PM, Cyrille Berger <cberger@cberger.net> wrote:

> On Friday 23 October 2009, Dmitry Kazakov wrote:
> > PS:
> > /me still thinks that using strings for internally used constants is a
> bad
> > practice =)
> > And /me doesn't think implicit sharing of QString will excuse it.
> > But i wont argue about it this time =)
> The problem is that those strings are not internal, and we want the set of
> composite op to be extensible. That's why we moved from int to qstring. As
> as
> side note, since QString (even with implicite sharing) comparison can be
> extensive, there is a class in pigment that gives you an integer for a
> string
> that can be stored to get fast comparison, we do that for comparing to
> colorspace. Here it would be an improvement, but the real fix is to migrate
> the
> remaining composite op.
>

Why ints are not extensible?


-- 
Dmitry Kazakov

[Attachment #5 (text/html)]

<br><br><div class="gmail_quote">On Fri, Oct 23, 2009 at 4:52 PM, Cyrille Berger \
<span dir="ltr">&lt;<a \
href="mailto:cberger@cberger.net">cberger@cberger.net</a>&gt;</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 Friday 23 \
October 2009, Dmitry Kazakov wrote:<br> &gt; PS:<br>
&gt; /me still thinks that using strings for internally used constants is a bad<br>
&gt; practice =)<br>
&gt; And /me doesn&#39;t think implicit sharing of QString will excuse it.<br>
&gt; But i wont argue about it this time =)<br>
</div>The problem is that those strings are not internal, and we want the set of<br>
composite op to be extensible. That&#39;s why we moved from int to qstring. As as<br>
side note, since QString (even with implicite sharing) comparison can be<br>
extensive, there is a class in pigment that gives you an integer for a string<br>
that can be stored to get fast comparison, we do that for comparing to<br>
colorspace. Here it would be an improvement, but the real fix is to migrate the<br>
remaining composite op.<br></blockquote><div><br>Why ints are not \
extensible?</div></div><br clear="all"><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]