From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Wed, 19 Jun 2019 11:17:54 GMT
Subject: Re: [PATCH 03/17] cache-tree.c: avoid reusing the DEBUG constant
Message-ID: <nycvar.QRO.7.76.6.1906191316210.44@tvgsbejvaqbjf.bet>
In-Reply-To: <CAPUEspi-G7abt-zuLbw0iuyXN0gtrM_6Vy6d+9f-=6FpRsoV4A@mail.gmail.com>

Hi Carlo,

On Tue, 18 Jun 2019, Carlo Arenas wrote:

> shouldn't this also be a problem with builtin/blame.c?

Sharp eyes!

It does not *really* matter as much here, as that file defines that
`DEBUG` constant if it has not yet been defined, but yes, it assumes that
*if* it is defined, then it is set to `1`. Which is the case, but it is
fragile.

I changed it locally already, and it will be part of the next iteration.

Thanks,
Johannes