Pidgin Improvement

[Richard Laager]

Sean,

You changed some of this code in this revision:
|   Revision: 73447ec367c4ed05325645f0717ff815e22ce3e6
|   Ancestor: 391beabbbe9b94205d5c2d9ef073ecff207da387
|   Author: seanegan at pidgin.im
|   Date: 2007-07-27T00:05:40
|   Branch: im.pidgin.pidgin
|   
|   Modified files:
|           libpurple/idle.c
|   
|   ChangeLog: 
|   
|   Fix idle reporting as discussed on devel at . Poll as needed when we're
not idle, and poll frequently (once every three seconds) when we are.
Fixes #1357

I'm no longer sure what is going on in this code. Can you shed some
light on this?

Richard

On Sun, 2007-09-23 at 01:29 -0700, Bz1 F wrote:
> Hello,
> 
> I was somewhat confused about an option on pidgin, and googled it,
> finding your discussion:
> 
> > On Wed, 2007-07-25 at 16:39 -0700, Sean Egan wrote:
> > > 1. The preference for "how long before going idle," is effectively
> 
> > > useless
> > I didn't realize we had a pref for this when I was looking at that code.
> 
> It was actually "how long before going auto-away as a result of idle,"
> so I can understand the confusion. I changed the preference to mean
> 
> "how long before going idle," which seems to make much more sense.
> 
> 
> 
> In fact, this is still confusing, as follows:
> 
> Currently it says: "Minutes before becoming Idle"
> 
> However, this function is supposed to set how many minutes to wait
> until the change of status to away AFTER already idle. 
> 
> Thus how can you become idle again after already idle.
> 
> I propose the following replacement: "Minutes idle before changing
> status"
> 
> This will make this much more clear.
> 
> Thanks,
> Ben
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://pidgin.im/pipermail/devel/attachments/20070927/20196993/attachment.sig>