Actions
Bug #714
closedgethostbyname() is obsolete and should be replaced by getaddrinfo()
Start date:
2017-01-27
Due date:
% Done:
100%
Estimated time:
Module:
dcmnet
Operating System:
all
Compiler:
Description
Actions
Added by Jörg Riesmeier over 8 years ago. Updated almost 8 years ago.
100%
Description
I'm not sure how much value there is in replacing one function with the other.
Keeping the current API and just emulating what gethostbyname() does using getaddrinfo() is somewhat cumbersome, and it doesn't really get us anything.
The chances that gethostbyname() vanishes in an OS release are, I think, very small.
The other option is to expose more-or-less the exact API of the new function as an addition to ofstd, analogous to the existing wrapper of gethostbyname().
However, since its functionality cannot be emulated by the older function, that would mandate the existence of getaddrinfo() for all DMCTK targets; I don't know if that is reasonable.
(This also applies to #715)
Closed by commit #77911e6 (public DCMTK) and #0c2f405 (private modules).