Re: [Sugar-devel] [PATCH] Infoslicer: Avoid crash when a article is downloaded - SL #2856

2011-05-20 Thread Walter Bender
On Thu, May 19, 2011 at 10:26 AM, Gonzalo Odiard godi...@sugarlabs.orgwrote: --- book.py |5 - bookview.py |1 - 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/book.py b/book.py index 681bdba..f28ff99 100644 --- a/book.py +++ b/book.py @@ -71,11 +71,14 @@

Re: [Sugar-devel] [PATCH] Infoslicer: Avoid crash when a article is downloaded - SL #2856

2011-05-20 Thread Gonzalo Odiard
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