[PATCH 1/1] builtin/checkout: simplify metadata initialization
- From
brian m. carlson <sandals@crustytoothpaste.net>
- Date
- May 20, 2020, 01:41 UTC
- Message-ID
- <20200520014156.1570124-1-sandals@crustytoothpaste.net>
- In-Reply-To
- <ef72aedf-4264-e386-9563-050c54483c93@gmail.com>
When we call init_checkout_metadata in reset_tree, we want to pass the object ID of the commit in question so that it can be passed to filters, or if there is no commit, the tree. We anticipated this latter case, which can occur elsewhere in the checkout code, but it cannot occur here, since reset_tree is called only (indirectly) via switch_branches, which requires that we have a valid commit. switch_branches dies if we lack a name and cannot produce a commit from HEAD, and its caller dies if we do have a branch name but still lack a commit pointer.
Since we know we must always have a valid commit structure in this case, let's remove the dead code paths and just refer to the commit structure. This simplifies the code and makes it easier for the reader.
Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net> --- builtin/checkout.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/builtin/checkout.c b/builtin/checkout.c index e9d111bb83..e53bdab5b8 100644 --- a/builtin/checkout.c +++ b/builtin/checkout.c @@ -620,11 +620,7 @@ static int reset_tree(struct tree *tree, const struct checkout_opts *o, opts.verbose_update = o->show_progress; opts.src_index = &the_index; opts.dst_index = &the_index; - init_checkout_metadata(&opts.meta, info->refname, - info->commit ? &info->commit->object.oid : - is_null_oid(&info->oid) ? &tree->object.oid : - &info->oid, - NULL); + init_checkout_metadata(&opts.meta, info->refname, &info->commit->object.oid, NULL); parse_tree(tree); init_tree_desc(&tree_desc, tree->buffer, tree->size); switch (unpack_trees(1, &tree_desc, &opts)) {