From: Benno <fw...@ro...> - 2013-03-02 18:37:18
|
On 28.02.2013 22:17, Daniele Forsi wrote: > 2013/2/28 phantomjinx: >> On 02/15/2013 06:16 PM, Daniele Forsi wrote: >>> 2013/2/15 Benno: >>> >>>> The reason was the comparison of different sized(at least in 64 bit systems) >>>> unsigned types, that were set to "-1" >>>> 1)in _NormalSortTabPagePrivate >>>> guint selection_changed_idle_id; >>>> 2)in _st_selection_changed() >>>> gulong idleId >>> >>> I committed your fix >>> http://sourceforge.net/mailarchive/message.php?msg_id=30490651 >>> >>> but now I think I misunderstood and I think I should revert it and >>> change idelId to guint instead because struct member >>> selection_changed_idle_id is assigned the return value from >>> gdk_threads_add_idle_full() which indeed is a guint. What do you >>> think? >>> >> >> Hey Daniele, >> >> Did anything come from this in the end? > > thank you for reminding this, hopefully now I have committed a fix > changing the type of the variable and the value of > NO_IDLE_SELECTION_CALLBACK to zero which according to the docs is > never returned by gdk_threads_add_idle_full () > I've just tested the patch on my 64bit system. It works! Thanks! |