Re: timevar: 1/3: expect that getrusage is available

2018-10-12 Thread Bruno Haible
Hi Akim, > > - The code prints the '.wall' values, which are not initialized. But this > >is fixed in part 3/3. > > They are initialized > > static void > set_to_current_time (struct timevar_time_def *now) > { > now->user = 0; > now->sys = 0; > now->wall = 0; > > but yes, they are

Re: timevar: 1/3: expect that getrusage is available

2018-10-12 Thread Akim Demaille
> Le 12 oct. 2018 à 17:27, Bruno Haible a écrit : > > Hi Akim, > > This patch looks good, except for two things: > - The module description still references m4/timevar.m4, which no longer >exists. Fixed, thanks! > - The code prints the '.wall' values, which are not initialized. But

Re: timevar: 1/3: expect that getrusage is available

2018-10-12 Thread Bruno Haible
Hi Akim, This patch looks good, except for two things: - The module description still references m4/timevar.m4, which no longer exists. - The code prints the '.wall' values, which are not initialized. But this is fixed in part 3/3. Bruno

Re: timevar: 1/3: expect that getrusage is available

2018-10-11 Thread Akim Demaille
Waiting for approval before pushing. commit 65b74a02c39450878ac1c41b870e1188c4a0429e Author: Akim Demaille Date: Thu Oct 11 17:54:35 2018 +0200 timevar: expect that getrusage is available Don't keep both times and getrusage as backend: both are guaranteed by gnulib, a single