<URL: http://bugs.freeciv.org/Ticket/Display.html?id=21635 >
> [book - Do 05. Okt 2006, 22:23:14]:
> ----- Original Message Follows -----
> From: "Christian Prochaska"
> <[EMAIL PROTECTED]>
> To: [EMAIL PROTECTED]
> Subject: (PR#21635) PATCH - asynchronous DNS core (and a few
> other things...)
> Date: Thu, 5 Oct 2006 08:49:49 -0700
> > <URL: http://bugs.freeciv.org/Ticket/Display.html?id=21635
> > >
> > > [book - Do 05. Okt 2006, 01:29:44]:
> > >
> > > - my_nonblock made to work on win32 (by using
> > > ioctlsocket if HAVE_WINSOCK is defined).
> > >
> > I made a similar patch in PR#19481 which covers some more
> > places where errno got evaluated after a socket function
> > call (which doesn't work with Winsock). Perhaps you'd like
> > to merge these additional changes into your netintf patch?
> It is easy now to change the places where errno is checked
> to the appropriate
> wrapper (my_socket_would_block,
> my_socket_operation_in_progess, my_errno).
> I'm not sure about the places EINTR is checked for, is it
> really ok to ignore that
> error? (If yes I suppose my_socket_operation_was_interrupted
> or some such should be
> As for checking errno for printing an appropriate message
> mystrerror should be used,
> or a similar translation function (that would do the right
> thing depending on the
> I think wrapper functions would be a better approach than
> besprinkling the code with
> #ifdefs, of course you maintainers know best :).
Yes, I agree. "Merge" was the wrong word. I actually meant integrating
the additional changes in a way that fits to the rest of your patch,
i.e. by making use of your new wrapper functions :-)
> By the way, what is ftwl?
It's a themeable widget library that can be used with different graphic
and input backends (currently supported backends are SDL and plain X11).
There's also a client (gui-ftwl) which makes use of the library, but
it's still in a very early development stage. You can find more
information about it in RT's client-ftwl queue, starting with PR#7123.
Freeciv-dev mailing list