Re: [PATCH] MI: new timing command

Nick Roberts
2006-12-30T22:10:22Z

> > Here's a patch for timing MI commands e.g.
>
> Thanks, but where's the documentation to go with it?

There isn't any yet. I would include it if I was confident that the patch
would get timely review/approval. I think the code part is easily understood
so I guess I'm asking for approval for that, conditional to tests/documentation
being provided. This saves wasted effort on my part.

> > diff -c -p -r1.86 mi-main.c
> > *** mi-main.c 17 Nov 2006 19:30:41 -0000 1.86
> > --- mi-main.c 30 Dec 2006 08:42:29 -0000
> > ***************
> > *** 49,54 ****
> > --- 49,55 ----
> >
> > #include <ctype.h>
> > #include <sys/time.h>
> > + #include <sys/resource.h>
>
> Is sys/resource.h guaranteed to be available on all supported
> platforms? If not, we need to add a configure-time test and enable
> this code only under HAVE_GETRUSAGE or some such.

mi-main.c already uses sys/time.h. Is sys/resource.h any less widely
distributed? Is there any way of finding out e.g online database?

--
Nick http://www.inet.net.nz/~nickrob