RE: [PATCH v6 08/12] fsmonitor: add a test tool to dump the index extension

> -Original Message- > From: Torsten Bögershausen [mailto:tbo...@web.de] > Sent: Tuesday, September 19, 2017 10:16 AM > To: Ben Peart > Cc: Ben Peart ; Junio C Hamano > ; david.tur...@twosigma.com; ava...@gmail.com; >

Re: [PATCH v6 08/12] fsmonitor: add a test tool to dump the index extension

> > Should I just make the variable type itself uintmax_t and then just skip > the cast altogether? I went with uint64_t because that is what > getnanotime returned. > That is a bit of taste question (or answer) Typically you declare the variables in the type you need, and this is uint64_t.

RE: [PATCH v6 08/12] fsmonitor: add a test tool to dump the index extension

> -Original Message- > From: Torsten Bögershausen [mailto:tbo...@web.de] > Sent: Monday, September 18, 2017 11:43 AM > To: Ben Peart ; Junio C Hamano > ; Ben Peart > Cc: david.tur...@twosigma.com; ava...@gmail.com; >

Re: [PATCH v6 08/12] fsmonitor: add a test tool to dump the index extension

On 2017-09-18 15:38, Ben Peart wrote: > > > On 9/17/2017 4:02 AM, Junio C Hamano wrote: >> Ben Peart writes: >> >>> diff --git a/t/helper/test-dump-fsmonitor.c b/t/helper/test-dump-fsmonitor.c >>> new file mode 100644 >>> index 00..482d749bb9 >>> --- /dev/null

Re: [PATCH v6 08/12] fsmonitor: add a test tool to dump the index extension

On 9/17/2017 4:02 AM, Junio C Hamano wrote: Ben Peart writes: diff --git a/t/helper/test-dump-fsmonitor.c b/t/helper/test-dump-fsmonitor.c new file mode 100644 index 00..482d749bb9 --- /dev/null +++ b/t/helper/test-dump-fsmonitor.c @@ -0,0 +1,21 @@ +#include

Re: [PATCH v6 08/12] fsmonitor: add a test tool to dump the index extension

Ben Peart writes: > diff --git a/t/helper/test-dump-fsmonitor.c b/t/helper/test-dump-fsmonitor.c > new file mode 100644 > index 00..482d749bb9 > --- /dev/null > +++ b/t/helper/test-dump-fsmonitor.c > @@ -0,0 +1,21 @@ > +#include "cache.h" > + > +int cmd_main(int