From: Mohit Marathe <mohitmarathe@proton.me>
Date: Wed, 17 Jan 2024 05:28:36 GMT
Subject: Re: [GSoC][RFC] Replace use of atoi() with strtol_i(), as a microproject
Message-ID: <Ec-xbd5ISDrcz8ybM8oJkVEhX7hgEmENoeek1n4K5M9buOfQ1SwOgGqAZRyMnYGYY1AFG1ppUKwXi3V63dndTzAjnJj7Cs0cVrtqwROg6oc=@proton.me>
In-Reply-To: <xmqqa5pdav04.fsf@gitster.g>

> A corrupt patch may be getting a nonsense patch-ID with the current
> code and hopefully is not matching other patches that are not
> corrupt, but with such a change, a corrupt patch may not be getting
> any patch-ID and a loop that computes patch-ID for many files and
> try to match them up might need to be rewritten to take the new
> failure case into account.

Are you talking about the loop in `get_one_patchid()`? And how should
the failure case be handled? Should it give a warning or an error?