Remove hip_in6_ntop and use inet_ntop instead

[Samu Varjonen]

hip_in6_ntop produces non-RFC3849-compliant IPv6 addresses. Does not remove leading zeroes. The function seems to be a leftover from the kernel times anyway. I cannot see a reason why not to use inet_ntop instead.

[Samu Varjonen (sklvarjo)]

Or RFC 5952? Anyway the leading zeroes are useless and the point was that the hip_in6_ntop is useless.

[René Hummen (rene-hummen)]

I agree. Samu, please go ahead and fix the issue. Please make sure to remove all references (e.g., in the unit-tests) to hip_in6_ntop. Thanks.

[Samu Varjonen (sklvarjo)]

I will remove them as soon as possible. Is there any objections if I remove the hip_convert_string_to_address on the same go and replace it with inet_pton? Used only in three places in hipconf. This would remove the lib/core/straddr.c(h) files and the test/lib/core/straddr.c file as useless.

[Miika Komu (miika-iki)]

Go ahead. Some developers might prefer two different (atomic) commits for the separate macros.

[Samu Varjonen (sklvarjo)]

OK, removing hip_convert_string_to_address as a separate commit...