Re: [PATCH v4 0/9] Introduce timestamp_t for timestamps

2017-04-25 Thread Johannes Schindelin
Hi Peff, On Mon, 24 Apr 2017, Jeff King wrote: > On Sun, Apr 23, 2017 at 08:29:11PM -0700, Junio C Hamano wrote: > > > Johannes Schindelin writes: > > > > > Changes since v3: > > > > > > - fixed the fix in archive-zip.c that tried to report a too large > > > timestamp (and would have reporte

Re: [PATCH v4 0/9] Introduce timestamp_t for timestamps

2017-04-24 Thread Johannes Schindelin
Hi Jacob, On Sun, 23 Apr 2017, Jacob Keller wrote: > On Sun, Apr 23, 2017 at 8:29 PM, Junio C Hamano wrote: > > Johannes Schindelin writes: > > > >> Changes since v3: > >> > >> - fixed the fix in archive-zip.c that tried to report a too large > >> timestamp (and would have reported the uninit

Re: [PATCH v4 0/9] Introduce timestamp_t for timestamps

2017-04-24 Thread Johannes Schindelin
Hi Junio, On Sun, 23 Apr 2017, Junio C Hamano wrote: > Johannes Schindelin writes: > > > Changes since v3: > > > > - fixed the fix in archive-zip.c that tried to report a too large > > timestamp (and would have reported the uninitialized time_t instead) > > > > - adjusted the so-far forgotten

Re: [PATCH v4 0/9] Introduce timestamp_t for timestamps

2017-04-24 Thread Jeff King
On Sun, Apr 23, 2017 at 08:29:11PM -0700, Junio C Hamano wrote: > Johannes Schindelin writes: > > > Changes since v3: > > > > - fixed the fix in archive-zip.c that tried to report a too large > > timestamp (and would have reported the uninitialized time_t instead) > > > > - adjusted the so-far

Re: [PATCH v4 0/9] Introduce timestamp_t for timestamps

2017-04-23 Thread Jacob Keller
On Sun, Apr 23, 2017 at 8:29 PM, Junio C Hamano wrote: > Johannes Schindelin writes: > >> Changes since v3: >> >> - fixed the fix in archive-zip.c that tried to report a too large >> timestamp (and would have reported the uninitialized time_t instead) >> >> - adjusted the so-far forgotten each_

Re: [PATCH v4 0/9] Introduce timestamp_t for timestamps

2017-04-23 Thread Junio C Hamano
Johannes Schindelin writes: > Changes since v3: > > - fixed the fix in archive-zip.c that tried to report a too large > timestamp (and would have reported the uninitialized time_t instead) > > - adjusted the so-far forgotten each_reflog() function (that was > introduced after v1, in 80f2a6097

[PATCH v4 0/9] Introduce timestamp_t for timestamps

2017-04-21 Thread Johannes Schindelin
Git v2.9.2 was released in a hurry to accomodate for platforms like Windows, where the `unsigned long` data type is 32-bit even for 64-bit setups. The quick fix was to simply disable all the testing with "absurd" future dates. However, we can do much better than that, as we already make use of 64