rxi/dyad — pull request #5 by dansgithubuser (2014-09-24 22:07:48+00:00)
Title: Fixed a crash on Windows at least; changed magic address length numbers ...

...to #defines

Hey, I experienced a crash on Windows 7 because of these magic numbers. According to this page:
http://pubs.opengroup.org/onlinepubs/009695399/functions/inet_ntop.html
inet_ntop's behavior can be bad if the size parameter is less than INET_ADDRSTRLEN for ipv4, INET6_ADDRSTRLEN for ipv6. So I just stuck those macros in.

I've tested on Windows 7, iOS 7, Mac OSX, and Android 4.4.2 at least. Not that you should trust me!

I'm not sure if you had a reason to use magic numbers instead of macros, maybe they are missing on some setups. I can do some #ifdef logic if that's the case though.

Best,
Dan