<div dir="ltr"><div>Great work Christian, sounds like this was a subtle one.</div><div><br></div>I know i was somehow involved at the start of this problem (but i can't remember/find how or when). Were these the random crashes especially on closing a PySide gui?<div>
<br></div><div>Anyway, do i understand correctly that i have to compile PySide to be able to use this patch or is it likely to show up in a release version of PySide?</div><div><br></div><div>Cheerio, Lars</div><div><br></div>
<div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Oct 21, 2013 at 7:07 PM, Christian Tismer <span dir="ltr"><<a href="mailto:tismer@stackless.com" target="_blank">tismer@stackless.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 21.10.13 18:57, John Ehresman wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 10/21/13 12:44 PM, Christian Tismer wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 21.10.13 17:54, John Ehresman wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I took a look at the patch and can't quite understand it (I must admit<br>
I didn't apply it).  Can someone explain why it's needed?<br>
</blockquote>
<br>
It is all about stackless' different frame layout.<br>
This is the minimal patch to be able to compile pyside for stackless<br>
python.<br>
It is explained in the bug report.<br>
<br>
<a href="https://bugreports.qt-project.org/browse/PYSIDE-199" target="_blank">https://bugreports.qt-project.<u></u>org/browse/PYSIDE-199</a><br>
</blockquote>
<br>
I think I'm confused -- I see changes to how SbkTypeObject fields are referenced (SbkTypeObject is a C level extension of the PyHeapTypeObject struct).  I don't think I see changes to frames.<br>
</blockquote>
<br></div>
Hoppla, you are right, no frames at all. Did I say that? geee<br>
<br>
No, it is about the type layout, there are (ah, here came the word frame<br>
again?! ) there are type fields in CPython that stackless does not have.<br>
<br>
Well, and because these structures are different in size, pyside must<br>
be compiled using stackless, otherwise it crashes ugly.<br>
<br>
(bzzt, no frames -- what's up with my brain? like a shortcut, I need rewiring)<div class="im HOEnZb"><br>
<br>
ciao - chris<br>
<br>
-- <br>
Christian Tismer             :^)   <mailto:<a href="mailto:tismer@stackless.com" target="_blank">tismer@stackless.com</a>><br>
Software Consulting          :     Have a break! Take a ride on Python's<br>
Karl-Liebknecht-Str. 121     :    *Starship* <a href="http://starship.python.net/" target="_blank">http://starship.python.net/</a><br>
14482 Potsdam                :     PGP key -> <a href="http://pgp.uni-mainz.de" target="_blank">http://pgp.uni-mainz.de</a><br>
phone <a href="tel:%2B49%20173%2024%2018%20776" value="+491732418776" target="_blank">+49 173 24 18 776</a>  fax <a href="tel:%2B49%20%2830%29%20700143-0023" value="+49307001430023" target="_blank">+49 (30) 700143-0023</a><br>

PGP 0x57F3BF04       9064 F4E1 D754 C2FF 1619  305B C09C 5A3B 57F3 BF04<br>
      whom do you want to sponsor today?   <a href="http://www.stackless.com/" target="_blank">http://www.stackless.com/</a><br>
<br>
<br></div><div class="HOEnZb"><div class="h5">
______________________________<u></u>_________________<br>
Stackless mailing list<br>
<a href="mailto:Stackless@stackless.com" target="_blank">Stackless@stackless.com</a><br>
<a href="http://www.stackless.com/mailman/listinfo/stackless" target="_blank">http://www.stackless.com/<u></u>mailman/listinfo/stackless</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>====================================<br>Lars van Gemerden<br><a href="mailto:lars@rational-it.com">lars@rational-it.com</a><br>+31 6 26 88 55 39<br>
====================================
</div>