[Pidgin] #393: Add receive Handwritten messages, Winks and Voice Clips to MSN

Pidgin trac at pidgin.im
Wed Sep 19 11:53:44 EDT 2007


#393: Add receive Handwritten messages, Winks and Voice Clips to MSN
--------------------+-------------------------------------------------------
  Reporter:  notak  |       Owner:  khc                 
      Type:  patch  |      Status:  assigned            
  Priority:  minor  |   Milestone:                      
 Component:  MSN    |     Version:  2.0                 
Resolution:         |    Keywords:  msn handwritten wink
   Pending:  0      |  
--------------------+-------------------------------------------------------
Comment (by notak):

 Tried to get this resurrected the other day when I finally updated to the
 latest P14 code, and I couldn't even get it to comile :-(. If I can't
 figure that much out I'm tempted to remove the winks anyway. As a side
 observation, these are delivered dreadfully slowly with direct
 connections:[[BR]]
 '''The audio stuff''' is basically quite cleanly implemented. My main
 issue is that in linux it isn't going to be too useful until the relevant
 codec gets included into normal codec packs. Until then I think the option
 is useful, since it will reject the attempt to send an audio clip at the
 other end, and stop the sender getting the impression that the receiver
 has received the message, so I'm quite keen on retaining the protocol
 option.[[BR]]
 '''The handwritten stuff''' is unfortunately pretty horribly hacked
 together by MSN - implementing them in multi-user chats is fairly simple
 as it just involves implementing the multi-part messaging code.
 Implementing them in chats however involves dealing with the fact that
 they are sent as slp messages with no attached msnobj. The message itself
 is sent in UTF16, in spite of the data being base64 encoded, and is
 separated from the header information by a \0. Personally I'd say it's a
 toss up deciding whether they wrote it that way to obfuscate the data, or
 they just really are that stupid, but whatever, the code required to
 decode is going to look pretty moronic. the +7 by the way is just to lose
 the text 'BASE64:' from the start of the data. I guess that much could be
 commented.[[BR]]
 [[BR]]
 If you are happy enough for me to separate out the winks and leave them
 for now, I'll try to put in a couple of comments to explain the magic code
 - hopefully tomorrow, depending on whether I can get the dev environment
 set up

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


More information about the Tracker mailing list