Re: fix for sorttab bug-no selecting at all possible(probably 64 bit issue)

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!

------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_feb