[Pidgin] #220: Pidgin does not display pictures recieved from Gadu Gadu

Pidgin trac at pidgin.im
Thu May 29 10:09:54 EDT 2008


#220: Pidgin does not display pictures recieved from Gadu Gadu
------------------------+---------------------------------------------------
  Reporter:  MarkS      |       Owner:  bartosz
      Type:  patch      |      Status:  new    
  Priority:  minor      |   Milestone:         
 Component:  Gadu-Gadu  |     Version:  2.0    
Resolution:             |    Keywords:         
   Pending:  0          |  
------------------------+---------------------------------------------------
Comment (by ono):

 @sabby7890: Please put your patch as single file as described in
 TipsForPatchSubmissions

 Regarding your latest patch:

 1. `msg = charset_convert(msg, "UTF-8", "CP1250");` and some other
 `g_strdup_printf` in `ggp_send_im_richtext` are IMHO memory leaks, since
 you don't free anywhere old `msg` memory buffer.

 2. UTF-8 -> CP1250 should happen at the very end just before sending the
 buffer to `libgadu` function, as all glib functions such as `g_strsplit`
 work in UTF-8 locale, so doing conversion to CP1250 first then working
 with UTF-8 aware functions if pretty bad idea since it may lead to
 unpredictable behavior.

 Anyway I really appreciate your work and looking towards putting it into
 repository once all the memory related things are sorted out.

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


More information about the Tracker mailing list