RE: nanosleep.c & winsock.h (was Re: Windows Build Broken - Feature Branch)

[Conrad T. Pino]

Hi Derek,

> From: Derek Robert Price
> 
> I think I'm past that point.  I just need sleep.  :)

Sounds like a good idea! :)

> >The hard issue is "nanosleep" needs a "struct timeval" definition in
> >order to make the "select" call and a redefinition error occurs when
> >"winsock.h" follows "unistd.h" in the same compile.
> 
> Ah.  You could try getting just declaring struct timeval in
> nanosleep.c past Jim Meyering.  Probably only when the other headers
> aren't found, of course.  And assuming it works.  Try it first...
============================
Here are the technical answers:

1. Adding a "struct timeval" definition in "nanosleep" does work.

2. "int select( );" alone is an unresolved external "_select" link error.
The minimum required to fix link error is "int _stdcall select( );".

3. An "_stdcall" function clears stack rather than the "C" way and I'm
assuming if the arguments are right there won't be a problem but this
is an untested assumption.
============================
Opinions follow:

I'd like to see CVS avoid no argument prototypes.

Since there's more in play than technical issues IMO the CVS project
is better served if you follow up with Jim so I leave this in your
good care.
============================
> Derek

Conrad