From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Sun, 06 Oct 2019 20:38:26 GMT
Subject: Re: [PATCH v3 00/13] ci: include a Visual Studio build & test in our Azure Pipeline
Message-ID: <nycvar.QRO.7.76.6.1910062237440.46@tvgsbejvaqbjf.bet>
In-Reply-To: <nycvar.QRO.7.76.6.1910061157320.46@tvgsbejvaqbjf.bet>

Hi Junio,

On Sun, 6 Oct 2019, Johannes Schindelin wrote:

> On Sun, 6 Oct 2019, Junio C Hamano wrote:
>
> > "Johannes Schindelin via GitGitGadget" <gitgitgadget@gmail.com>
> > writes:
> >
> > > Range-diff vs v2:
> > >
> > >   1:  4d0b38125a =  1:  4d0b38125a push: do not pretend to return `int` from `die_push_simple()`
> > >   2:  8800320590 <  -:  ---------- msvc: avoid using minus operator on unsigned types
> > >   -:  ---------- >  2:  7fe2a85506 msvc: avoid using minus operator on unsigned types
> > >   3:  8512a3e96d =  3:  e632a4eef4 winansi: use FLEX_ARRAY to avoid compiler warning
> >
> > This is less useful than it could be.
> >
> > With a larger creation-factor (and we can afford using a larger one,
> > simply because the user of GGG _knows_ that the two series being
> > compared are closely related), what is output is entirely readable
> > (attached at the end).
>
> I just implemented this here:
> https://github.com/gitgitgadget/gitgitgadget/pull/128 (it still needs to
> be reviewed and merged before it takes effect).

FWIW this is now merged.

Ciao,
Dscho