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

List:       pykde
Subject:    Re: [PyKDE] Segfault in QGraphicsScene.drawItems
From:       "Arve Knudsen" <arve.knudsen () gmail ! com>
Date:       2007-02-25 14:30:31
Message-ID: a0d6258d0702250630m36a82b44u2761b55db9ec4277 () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


On 2/25/07, Phil Thompson <phil@riverbankcomputing.co.uk> wrote:
>
> On Sunday 25 February 2007 1:22 pm, Arve Knudsen wrote:
> > Now it's compiled, but I see that drawItems still receives the numItems
> > argument. Isn't this redundant considering that this is only the length
> of
> > the items list? Or is there a point in keeping it around to resemble the
> > C++ interface?
>
> There is no technical need to keep it - I was going for "least surprise".
> However, as you are obviously surprised then I failed :)
>
> I'll remove it in the next snapshot unless you disagree.
>

I don't know about others, but I don't see any compelling reason to keep
this parameter around in the Python interface. Thanks again. Hopefully I can
now finish porting the rest of this application prototype from C++ to Python
:)

Arve

[Attachment #5 (text/html)]

<br><br><div><span class="gmail_quote">On 2/25/07, <b class="gmail_sendername">Phil \
Thompson</b> &lt;<a href="mailto:phil@riverbankcomputing.co.uk">phil@riverbankcomputing.co.uk</a>&gt; \
wrote:</span><blockquote class="gmail_quote" style="margin-top: 0; margin-right: 0; \
margin-bottom: 0; margin-left: 0; margin-left: 0.80ex; border-left-color: #cccccc; \
border-left-width: 1px; border-left-style: solid; padding-left: 1ex"> On Sunday 25 \
February 2007 1:22 pm, Arve Knudsen wrote:<br>&gt; Now it&#39;s compiled, but I see \
that drawItems still receives the numItems<br>&gt; argument. Isn&#39;t this redundant \
considering that this is only the length of <br>&gt; the items list? Or is there a \
point in keeping it around to resemble the<br>&gt; C++ interface?<br><br>There is no \
technical need to keep it - I was going for &quot;least surprise&quot;.<br>However, \
as you are obviously surprised then I failed :) <br><br>I&#39;ll remove it in the \
next snapshot unless you disagree.<br></blockquote></div><br>I don&#39;t know about \
others, but I don&#39;t see any compelling reason to keep this parameter around in \
the Python interface. Thanks again. Hopefully I can now finish porting the rest of  \
this application prototype from C++ to Python :) <br><br>Arve<br>



_______________________________________________
PyKDE mailing list    PyKDE@mats.imk.fraunhofer.de
http://mats.imk.fraunhofer.de/mailman/listinfo/pykde


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