-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Jorge Arellano Cid wrote:
Not yet committed, mainly because the feature freeze for release. (although the small fix went in).
The attached patch fixes this and also adds the function
void a_DilloHost_to_String(DilloHost *host, char *dst, size_t size)
If you finally get to a complete fix, please name exported functions in dns.c with the "a_Dns_" prefix. e.g. a_Dns_host_to_string Sure thing. Updated patch is attached.
Patches are more than welcomed! And you shall have em ;)
While tracking my ipv6 related problems through dillos source files I noticed *a lot* of ENABLE_IPV6 #ifdefs... Is this necessary for portability or do you want to be able to strip ipv6 support from certain builds? If portability at compile time is not the issue I would vote for the removal of those #ifdefs, if there's worry about the executables size, let's see: $ wc --bytes with{,out}/dillo2/src/dillo-fltk 3311349 with/dillo2/src/dillo-fltk 3309974 without/dillo2/src/dillo-fltk Well, IPv6 costs 1375 bytes on my system, I don't see any problem here, but I think this could be reduced. Many of those ipv6 related #ifdefs duplicate quite a lot of code. One tiny little cleanup patch is attached (please note that it depends on the a_Dns_DilloHost_to_String function). Justus -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iD8DBQFI5/zFoPmwNWhsaZYRAiEIAJ9qNt+kRIN0khlpladZ3eD8r3fl6QCdHoCD VpRIfaqp6KA0Z6XgS+2wU2o= =39Iv -----END PGP SIGNATURE-----