-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Justus Winter wrote:
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. *bump*
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). Any thoughts on this one?
Justus -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iEYEARECAAYFAkkHFIAACgkQoPmwNWhsaZbaUQCfegvuOhWQgZw1Fgbv1WUklIRs j58AoKUT8xSrbIUR6gLLHfnGNQQ1km/0 =JjH6 -----END PGP SIGNATURE-----