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 it's

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. Would you

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-dwProcessId); +

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-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

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? Attached.