Re: [PATCH] GnuTLS support on Woe32

[Eli Zaretskii]

> From: address@hidden (Claudio Bley)
> Date: Sun, 13 Mar 2011 14:53:12 +0100
> 
> > If all you need is to produce EAGAIN when you have EWOULDBLOCK (the
> > other mapping is already in set_errno), it hardly justifies a
> > function.
> 
> That's true, WSAEINTR already gets mapped. Must have missed that.

So are we in agreement that a separate new function is not required?

> > > > > > +static ssize_t
> > > > > > +emacs_gnutls_pull(gnutls_transport_ptr_t p, void* buf, size_t sz)
> > > > > 
> > > > > Can we move the Windows-specific functions to w32.c, and only call
> > > > > them from gnutls.c?  I think we want to keep the Windows-related code
> > > > > outside w32*.c to the bare minimum.
> > > > 
> > > > OK.
> > > 
> > > Maybe the GnuTLS specific stuff should also be kept to the bare
> > > minimum outside of gnutls.c?
> > 
> > What stuff did you have in mind?
> 
> All the GnuTLS related functions (even if Windows specific).

That's not what we do in Emacs.  OS-specific #define's are best kept
to a minimum, the sole exception being sysdep.c.  Otherwise, we try to
keep code of non-Posix and niche platforms on their specific sources
files.

> > > Considering that these functions would have to be non-static in this
> > > case to be accessible by gnutls.c.
> > 
> > Sure, but I see no problem with that.
> 
> I'm usually a bit reluctant to create public functions in a module
> which only serve a special purpose in one single other module.

Why?  Emacs is a program, not a general-purpose library.  Invading
some unknown namespace should not be an issue.