wahjam/wahjam — pull request #57 by stefanha (2012-07-01 15:50:21+00:00)
Title: Switch from unlink(2) to remove(3) in njclient.cpp

The build fails on non-Windows platforms due to a unlink(2) call in
njclient.cpp without including unistd.h. It is not clear why this build
failure never appeared before but it was introduced in commit
71503ffdde2cd15f835f2acc968ce65105a27774 "Replace JNL_Connection with
QTcpSocket" when the jnetlib header was no longer included.

Since njclient.cpp already includes stdio.h, use the standard C
remove(3) function instead of POSIX unlink(2). This way we avoid more
platform-specific #ifdefs.

Signed-off-by: Stefan Hajnoczi