diff options
| author | Luke Shumaker <lukeshu@datawire.io> | 2021-04-27 15:17:41 -0600 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2021-04-28 16:47:18 +0900 |
| commit | 5cdae0f6fd76bd1c12304240a17a5a12632fbb12 (patch) | |
| tree | 816c65eb84377a75bcdd7f09d1ee76bb342e6157 /builtin/commit.c | |
| parent | subtree: remove duplicate check (diff) | |
| download | git-5cdae0f6fd76bd1c12304240a17a5a12632fbb12.tar.gz git-5cdae0f6fd76bd1c12304240a17a5a12632fbb12.zip | |
subtree: add comments and sanity checks
For each function in subtree, add a usage comment saying what the
arguments are, and add an `assert` checking the number of arguments.
In figuring out each thing's arguments in order to write those comments
and assertions, it turns out that find_existing_splits is written as if
it takes multiple 'revs', but it is in fact only ever passed a single
'rev':
unrevs="$(find_existing_splits "$dir" "$rev")" || exit $?
So go ahead and codify that by documenting and asserting that it takes
exactly two arguments, one dir and one rev.
Signed-off-by: Luke Shumaker <lukeshu@datawire.io>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/commit.c')
0 files changed, 0 insertions, 0 deletions
