<br><br><div class="gmail_quote">On Fri, May 20, 2011 at 1:15 PM, Gonzalo Odiard <span dir="ltr"><<a href="mailto:godiard@sugarlabs.org">godiard@sugarlabs.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br><div class="gmail_quote"><div class="im"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">I am not very familiar with treeview, but the "path" in the set_cursor call seems to be OK (just ran some tests) but maybe it is a race condition of some sort: pointing to the new last entry in the list before the list knows it has a new last entry? In any case, simply eliminating the call to set_cursor seems to be enough to address the crash problem. What is the first part of the patch trying to accomplish?<br>


<br></blockquote></div><div><br>I have seen a crash in Fedora 14 .<br>The error is the second in the ticket:<br><br><pre>1305794514.172503 DEBUG infoslicer: load article b5ae3528-81f3-11e0-873b-0017c4a748fe<br><br>
Gdk-ERROR **: The program 'sugar-activity' received an X Window System error.<br>This probably reflects a bug in the program.<br>The error was 'RenderBadPicture (invalid Picture parameter)'.<br>  (Details: serial 8423 error_code 159 request_code 147 minor_code 6)<br>

  (Note to programmers: normally, X errors are reported asynchronously;<br>   that is, you will receive the error a while after causing it.<br>   To debug your program, run it with the --sync command line<br>   option to change this behavior. You can then get a meaningful<br>

   backtrace from your debugger if you break on the gdk_x_error() function.)<br>aborting...<br>Terminated by signal 6, pid 3283 data (None, <open file '<fdopen>', mode 'w' at 0x8cd8f98>, dbus.ByteArray('76b6586b426d3aaa9a710551c59d77fae4d574e6', variant_level=1))<br>

</pre><pre>I could reproduce it several times.<br></pre>If you want, I can send you two different patches<br></div></div></blockquote><div><br>No need to send two separate patches... I was just curious what the relationship between the two was. I take it that the X bug had to do with timing, which the call to idle_add fixes?<br>
<br>-walter<br> <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="gmail_quote"><div>Gonzalo<br><br> </div><div class="im">
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
-walter<br><font color="#888888"><br>-- <br>Walter Bender<br>Sugar Labs<br><a href="http://www.sugarlabs.org" target="_blank">http://www.sugarlabs.org</a><br><br>
</font></blockquote></div></div><font color="#888888"><br><br clear="all"><br>-- <br>Gonzalo Odiard<br>SugarLabs Argentina<br><br>
</font></blockquote></div><br><br clear="all"><br>-- <br>Walter Bender<br>Sugar Labs<br><a href="http://www.sugarlabs.org">http://www.sugarlabs.org</a><br><br>