Re: [PATCH] CPU-time clocks

2011-05-17 Thread Corinna Vinschen
On May 17 04:10, Yaakov (Cygwin/X) wrote: > On Tue, 2011-05-17 at 07:58 +0200, Corinna Vinschen wrote: > > Thank you. You can apply it, but while I was looking into it, > > this occured to me: > > > > This conversion arithmetic from FILETIME to long long happens a lot > > in times.cc, even though

Re: [PATCH] CPU-time clocks

2011-05-17 Thread Yaakov (Cygwin/X)
On Tue, 2011-05-17 at 07:58 +0200, Corinna Vinschen wrote: > Thank you. You can apply it, but while I was looking into it, > this occured to me: > > This conversion arithmetic from FILETIME to long long happens a lot > in times.cc, even though it's absolutely not necessary. > > The FILETIME stru

Re: [PATCH] CPU-time clocks

2011-05-16 Thread Corinna Vinschen
On May 16 18:10, Yaakov (Cygwin/X) wrote: > On Mon, 2011-05-16 at 12:43 +0200, Corinna Vinschen wrote: > > Thanks for this patch. It looks good to me with two exceptions: > > [...] > Revised patch attached. Thank you. You can apply it, but while I was looking into it, this occured to me: > +

Re: [PATCH] CPU-time clocks

2011-05-16 Thread Yaakov (Cygwin/X)
On Mon, 2011-05-16 at 12:43 +0200, Corinna Vinschen wrote: > Thanks for this patch. It looks good to me with two exceptions: > > > extern "C" int > > clock_gettime (clockid_t clk_id, struct timespec *tp) > > { > > [...] > > + hProcess = OpenProcess (PROCESS_QUERY_INFORMATION, 0, > > p->d

Re: [PATCH] CPU-time clocks

2011-05-16 Thread Corinna Vinschen
Hi Yaakov, On May 15 14:31, Yaakov (Cygwin/X) wrote: > On Sun, 2011-05-15 at 21:11 +0200, Corinna Vinschen wrote: > > I just applied a patch to implement pthread_attr_setstack etc. > > Yes, I just saw that, thank you. > > > This affects your patch in that it won't apply cleanly anymore. > > Woul

Re: [PATCH] CPU-time clocks

2011-05-15 Thread Yaakov (Cygwin/X)
On Sun, 2011-05-15 at 21:11 +0200, Corinna Vinschen wrote: > I just applied a patch to implement pthread_attr_setstack etc. Yes, I just saw that, thank you. > This affects your patch in that it won't apply cleanly anymore. > Would you mind to regenerate your patches relative to CVS HEAD? Attache

Re: [PATCH] CPU-time clocks

2011-05-15 Thread Corinna Vinschen
Hi Yaakov, On May 15 13:37, Yaakov (Cygwin/X) wrote: > The attached patches implement POSIX CPU-time clock support: I just applied a patch to implement pthread_attr_setstack etc. This affects your patch in that it won't apply cleanly anymore. Would you mind to regenerate your patches relative to