Re: [PATCH 1/6] revision: use strtol_i() for exclude_parent

[PATCH 0/6] revision: fix order of revs for ^! · René Scharfe <hidden> · 2022-09-15
[PATCH 1/6] revision: use strtol_i() for exclude_parent · René Scharfe <hidden> · 2022-09-15
Re: [PATCH 1/6] revision: use strtol_i() for exclude_parent · Chris Torek <hidden> · 2022-09-17
[PATCH 2/6] revision: factor out get_commit() · René Scharfe <hidden> · 2022-09-15
[PATCH 3/6] revision: factor out add_parent() · René Scharfe <hidden> · 2022-09-15
[PATCH 4/6] revision: factor out add_parents() · René Scharfe <hidden> · 2022-09-15
[PATCH 5/6] revision: rename add_parents_only() to add_nth_parent() · René Scharfe <hidden> · 2022-09-15
[PATCH 6/6] revision: add parents after child for ^! · René Scharfe <hidden> · 2022-09-15
[PATCH v3 0/3] diff: support ^! for merges · René Scharfe <hidden> · 2022-10-01
[PATCH v3 1/3] revision: use strtol_i() for exclude_parent · René Scharfe <hidden> · 2022-10-01
[PATCH v3 2/3] revisions.txt: unspecify order of resolved parts of ^! · René Scharfe <hidden> · 2022-10-01
[PATCH v3 3/3] diff: support ^! for merges · René Scharfe <hidden> · 2022-10-01

From: Chris Torek <hidden>
Date: 2022-09-17 01:00:02

On Thu, Sep 15, 2022 at 7:51 AM René Scharfe [off-list ref] wrote:

Avoid silent overflow of the int exclude_parent by using the appropriate
function, strtol_i(), to parse its value.

Signed-off-by: René Scharfe <redacted>

Nice (albeit small) improvement regardless of anything else,

Chris

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help