Revision 45666c61a140448a2974f59f1bee801bdc8a4ab2
Evan Schoenberg
evan.s at dreskin.net
Sun Feb 25 16:14:11 EST 2007
On Feb 25, 2007, at 3:45 PM, Mark Doliner wrote:
>> Modified files:
>> libpurple/dnsquery.c
>>
>> ChangeLog:
>>
>> Return -1 as 'invalid port' if passed NULL in
>> gaim_dnsquery_get_port()
>
> It might be better to return an unsigned short and use 0 as
> 'invalid port'
> (that's what the code in network.c does). Also, some of your
> indentation in
> the UI DNS resolution commit was strangely inconsistent :-)
'k, I'll fix both of these in the near future.
Indentation problems are because I've uninstalled my normal coding
IDE (XCode) to force myself to do proper work until the week's end...
but then a conversation with Graham led to me doing some coding,
which I did in a different text editor that apparently defaults to
using spaces instead of tabs.
-Evan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://pidgin.im/pipermail/commits/attachments/20070225/eff49f80/attachment.htm
-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 186 bytes
Desc: This is a digitally signed message part
Url : http://pidgin.im/pipermail/commits/attachments/20070225/eff49f80/attachment.pgp
More information about the Commits
mailing list