On Mon, 10 Apr 2017 21:33:14 +0200 Christian Brauner <christian.brau...@ubuntu.com> wrote:
> The code uses kernel-style and it seems this is what screen master is > also aiming at. Hi, yes... but it makes it hard to review patch if you change code style while adding other changes. Can you resend without changing format. (On master all source code was reformatted in separate commit) One thing I wonder in particular is wouldn't it make everythink simpler to leave MakeClientSocket call declaration as it is and move socket check inside, but would like to get a better look at patch with less churn. Somethink like: int MakeClientSocket(int err) { bool socket; socket = IsSocket(SockPath); if (socket) return MakeClientUnixSocket(err); return MakeClientFifo(err); } Cheers, Amadeusz