From: Jon Simons <jon@jonsimons.org>
Date: Thu, 17 Oct 2019 23:53:44 GMT
Subject: Re: [PATCH v4 00/17] New sparse-checkout builtin and "cone" mode
Message-ID: <1ea216d8-e706-ec4c-a3f3-954e57e0458c@jonsimons.org>
In-Reply-To: <pull.316.v4.git.1571147764.gitgitgadget@gmail.com>

On 10/15/19 6:55 AM, Derrick Stolee via GitGitGadget wrote:
> V4 UPDATE: Rebased on latest master to include ew/hashmap and
> ds/include-exclude in the base.

I did a partial review of the v4 patches out of curiosity and I notice
in sparse-checkout.c there are a couple of unchecked `fopen` call sites
that could be converted to `xfopen` to die gracefully upon error, and
one unchecked `fdopen` that can likewise be `xfdopen`.


-Jon