[Pidgin] #8690: Add authorization request message to callback

Pidgin trac at pidgin.im
Mon Dec 27 21:01:07 EST 2010


#8690: Add authorization request message to callback
-----------------------+----------------------------------------------------
 Reporter:  cockroach  |        Owner:           
     Type:  patch      |       Status:  new      
Milestone:  2.8.0      |    Component:  libpurple
  Version:  2.5.5      |   Resolution:           
 Keywords:             |  
-----------------------+----------------------------------------------------

Comment(by deryni):

 I'd probably drop the explicit enum values as they don't matter, also I'd
 make the order _PASS, _IGNORE, _DENY, _ACCEPT.

 I'd probably also s/_IGNORE/_DROP/ (it seems more clear/appropriate to
 me).

 I'm also not a huge fan of _PASS but haven't come up with a better name
 yet (the best I've gotten is _PASSTHROUGH or _ASK or _DEFAULT).

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


More information about the Tracker mailing list