Re: [PATCH] Implement sigtimedwait (revised)

2017-12-18 Thread Corinna Vinschen
On Dec 14 17:05, Mark Geisert wrote: > Abstract out common code from sigwait/sigwaitinfo/sigtimedwait to implement > the latter. > --- > winsup/cygwin/common.din | 1 + > winsup/cygwin/include/cygwin/version.h | 3 ++- > winsup/cygwin/signal.cc| 36 > +

Re: [PATCH] Implement sigtimedwait (revised)

2017-12-18 Thread Corinna Vinschen
On Dec 18 11:07, Yaakov Selkowitz wrote: > On 2017-12-18 04:48, Corinna Vinschen wrote: > > On Dec 18 01:34, Mark Geisert wrote: > >> On Mon, 18 Dec 2017, Corinna Vinschen wrote: > >>> Hi Mark, > >> [...] > >>> as I wrote on Friday, the patch looks good to me. I just need a > >>> contributors lice

Re: [PATCH] Implement sigtimedwait (revised)

2017-12-18 Thread Yaakov Selkowitz
On 2017-12-18 04:48, Corinna Vinschen wrote: > On Dec 18 01:34, Mark Geisert wrote: >> On Mon, 18 Dec 2017, Corinna Vinschen wrote: >>> Hi Mark, >> [...] >>> as I wrote on Friday, the patch looks good to me. I just need a >>> contributors license agreement from you per the "Before you get started"

Re: [PATCH] Implement sigtimedwait (revised)

2017-12-18 Thread Corinna Vinschen
On Dec 18 01:34, Mark Geisert wrote: > On Mon, 18 Dec 2017, Corinna Vinschen wrote: > > Hi Mark, > [...] > > as I wrote on Friday, the patch looks good to me. I just need a > > contributors license agreement from you per the "Before you get started" > > section on https://cygwin.com/contrib.html >

Re: [PATCH] Implement sigtimedwait (revised)

2017-12-18 Thread Mark Geisert
On Mon, 18 Dec 2017, Corinna Vinschen wrote: Hi Mark, [...] as I wrote on Friday, the patch looks good to me. I just need a contributors license agreement from you per the "Before you get started" section on https://cygwin.com/contrib.html Hi Corinna, Y'all should have one from me on file al

Re: [PATCH] Implement sigtimedwait (revised)

2017-12-18 Thread Corinna Vinschen
Hi Mark, On Dec 14 17:05, Mark Geisert wrote: > Abstract out common code from sigwait/sigwaitinfo/sigtimedwait to implement > the latter. > --- > winsup/cygwin/common.din | 1 + > winsup/cygwin/include/cygwin/version.h | 3 ++- > winsup/cygwin/signal.cc| 36 > +++

Re: [PATCH] Implement sigtimedwait

2017-12-15 Thread Corinna Vinschen
On 15 December 2017 02:09:42 GMT+01:00, Mark Geisert wrote: >On Thu, 14 Dec 2017, Corinna Vinschen wrote: >> Hi Mark, >> >> Thanks for sigtimedwait! Two questions: >> >> On Dec 13 22:54, Mark Geisert wrote: >>> diff --git a/winsup/cygwin/signal.cc b/winsup/cygwin/signal.cc >>> index 69c5e2aad..05

Re: [PATCH] Implement sigtimedwait

2017-12-14 Thread Mark Geisert
On Thu, 14 Dec 2017, Corinna Vinschen wrote: Hi Mark, Thanks for sigtimedwait! Two questions: On Dec 13 22:54, Mark Geisert wrote: diff --git a/winsup/cygwin/signal.cc b/winsup/cygwin/signal.cc index 69c5e2aad..0599d8a3e 100644 --- a/winsup/cygwin/signal.cc +++ b/winsup/cygwin/signal.cc [...]

[PATCH] Implement sigtimedwait (revised)

2017-12-14 Thread Mark Geisert
Abstract out common code from sigwait/sigwaitinfo/sigtimedwait to implement the latter. --- winsup/cygwin/common.din | 1 + winsup/cygwin/include/cygwin/version.h | 3 ++- winsup/cygwin/signal.cc| 36 -- winsup/cygwin/thread.cc

Re: [PATCH] Implement sigtimedwait

2017-12-14 Thread Corinna Vinschen
Hi Mark, Thanks for sigtimedwait! Two questions: On Dec 13 22:54, Mark Geisert wrote: > diff --git a/winsup/cygwin/signal.cc b/winsup/cygwin/signal.cc > index 69c5e2aad..0599d8a3e 100644 > --- a/winsup/cygwin/signal.cc > +++ b/winsup/cygwin/signal.cc > [...] > + } > + cwaittime.QuadPart

[PATCH] Implement sigtimedwait

2017-12-13 Thread Mark Geisert
Abstract out common code from sigwait/sigwaitinfo/sigtimedwait to implement the latter. --- winsup/cygwin/common.din | 1 + winsup/cygwin/include/cygwin/version.h | 3 ++- winsup/cygwin/signal.cc| 32 ++-- winsup/cygwin/thread.cc