On Tue, Feb 6, 2018 at 5:12 PM, Brandon Williams <bmw...@google.com> wrote:
> Factor out the logic for processing shallow, deepen, deepen_since, and
> deepen_not lines into their own functions to simplify the
> 'receive_needs()' function in addition to making it easier to reuse some
> of this logic when implementing protocol_v2.
> Signed-off-by: Brandon Williams <bmw...@google.com>
Reviewed-by: Stefan Beller <sbel...@google.com>
for the stated purpose of just refactoring existing code for better reuse later.
I do have a few comments on the code in general,
which might be out of scope for this series.
A close review would have been fastest if we had some sort of
which I might revive soon for this purpose. (it showed that I would need it)
> + *depth = (int)strtol(arg, &end, 0);
strtol is not used quite correctly here IMHO, as we do not
inspect errno for ERANGE