[Pidgin] #5762: Fix mail handling

Pidgin trac at pidgin.im
Wed Jul 16 01:11:18 EDT 2008


#5762: Fix mail handling
----------------------+-----------------------------------------------------
  Reporter:  felipec  |       Owner:  QuLogic
      Type:  patch    |      Status:  new    
  Priority:  minor    |   Milestone:         
 Component:  MSN      |     Version:  2.4.1  
Resolution:           |    Keywords:         
   Pending:  0        |  
----------------------+-----------------------------------------------------
Comment (by felipec):

 Replying to [comment:4 QuLogic]:
 > Well, it works for me, but that's right at login. I assume this is for
 after that?

 Right.

 > 42075c looks reasonable. I assume it's because "URL" is treated as async
 and thus cmd->trans isn't set for that callback. Aren't we sending a
 "URL", though? Should this not be treated as async, or is there a chance
 we could get it unrequested?

 I believe at some point it was sent automatically, maybe not anymore.

 > If the right time is 777 seconds, then why does the code say 750? Is
 that "just in case"?

 Yes.

 > e80c23 makes sense as long as the rest does.
 >
 > That QNG result isn't used as a timeout anymore.
 >
 > I will try to commit this tomorrow, as it's too late for me to test now.

-- 
Ticket URL: <http://developer.pidgin.im/ticket/5762#comment:5>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list