Why `getnameinfo` function for macosX64 and linuxX...

# multiplatform

[aleksey.tomin]

Why getnameinfo function for macosX64 and linuxX64 target in platform.posix but for mingwX64 in platform.windows ?
It it a bug or feature?

[Artyom Degtyarev [JB]]

This is an expectable difference, as this function is located in different headers: ws2tcpip.h for mingw_x64 target and netdb.h for macOS/Linux. The first header is defined as a part of windows.def , while the second can be found in <osx|linux>/posix.def . While it can be relocated to the mingw’s posix KLIB, one could say it is controversial decision.

[aleksey.tomin]

I use “host” target - same as https://github.com/Kotlin/kotlinconf-spinner/tree/master/httpserver
And I’ve added getnameinfo function to allow requests only from localhost. And now I can’t compile same code on Win and Mac.
How can I do it? I can’t use 1.4-M3 because my app cann’t be compiled on this version.

[Artyom Degtyarev [JB]]

Hmm, that could be a problem.
Maybe this obstacle worth moving to more straightforward targeting, with separate source sets. If the signature is the same, one could try using expect/actual mechanism, typealiasing actual function to a platform-specific one.

[aleksey.tomin]

expect typealias Addr

Modifier 'expect' is not applicable to 'typealias'

[Artyom Degtyarev [JB]]

Sorry, I meant something like expect fun getAddr in the common code, with appropriate signature, and actual typealias getAddr = platform.posix.getaddrinfo

[aleksey.tomin]

expect fun getAddr(addr: CPointer<sockaddr>?) - type CPointer<sockaddr>? isn’t allowed in common code

Now I’ve created
expect fun getNameInfo(addrAsLong: Long, addrLength: UInt): String
and

if (getIdAddr(addr.toLong(), addrLen) == LOCALHOST) {
    MHD_YES
} else {
    MHD_NO
}

but it looks like ugly

[aleksey.tomin]

Oh…
MacOS: socklen_t - UInt
MinGW: socklen_t - Int
💩