[Pidgin] #10793: Proposed fix to ticket 3329 (Autocompletion of screennames with spaces)

Pidgin trac at pidgin.im
Sat Nov 21 19:48:49 EST 2009


#10793: Proposed fix to ticket 3329 (Autocompletion of screennames with spaces)
---------------------+------------------------------------------------------
Reporter:  mthurman  |        Type:  patch       
  Status:  new       |   Component:  pidgin (gtk)
 Version:  2.6.3     |    Keywords:              
---------------------+------------------------------------------------------
 This patch fixes the problem with tab completion as outlined in 3329 (a
 defect).  We essentially used the code that was already in the function
 and just move things around.  Being new, we didn't want to change too much
 but we don't see the need for both temp and nick_partial (even though we
 kept both).

 Since we can assume that entered has no white space, we solved the problem
 by stripping out spaces from the name we are comparing to. I'm not sure if
 '\t', '\r', '\n' (etc) are valid in names (I assume not) but they would
 still not be matched with this code.

 This patch is submitted by Mark Thurman (mthurman AT gmail DOT com) and
 Anthony Hugh (ahugh23 AT gmail DOT com).

-- 
Ticket URL: <http://developer.pidgin.im/ticket/10793>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list