On Sun, Sep 13, 2009 at 11:46 PM, Vitaliy Margolen <wine-devel(a)kievinfo.com> wrote:
Forgot this part:
Ismael Barros wrote: +static DWORD WINAPI tcp_listener_thread( LPVOID lpParameter ) +{ + for ( ;; ) + { + if ( clientSock == INVALID_SOCKET ) + { + goto end; + } + } + +end: +} It's more cleaner to use "break;" instead of goto to exit the loop.
It's a way to tell between abruptly ending the thread due to an error (so it just executes cleanup code and returns), and ending the thread gracefully (which would be done with a break or a thread termination boolean). In this case is equivalent because there's no code between the end of the loop and the "end" label. If it's better to use always a break I will change it.