diff options
author | Elijah Newren <newren@gmail.com> | 2023-02-27 15:28:14 +0000 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2023-02-27 08:29:51 -0800 |
commit | 1147c56ff70d5d1152601a73c9e76b9856ceebf6 (patch) | |
tree | 7c000d172f15fb1851c56993f1fb3826de8b3b5d /diff.c | |
parent | 5d4f4a592e99e8fb220b2db410f0d81203f3762e (diff) |
sparse-checkout: avoid using internal API of unpack-trees
struct unpack_trees_options has the following field and comment:
struct pattern_list *pl; /* for internal use */
Despite the internal-use comment, commit e091228e17 ("sparse-checkout:
update working directory in-process", 2019-11-21) starting setting this
field from an external caller. At the time, the only way around that
would have been to modify unpack_trees() to take an extra pattern_list
argument, and there's a lot of callers of that function. However, when
we split update_sparsity() off as a separate function, with
sparse-checkout being the sole caller, the need to update other callers
went away. Fix this API problem by adding a pattern_list argument to
update_sparsity() and stop setting the internal o.pl field directly.
Signed-off-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'diff.c')
0 files changed, 0 insertions, 0 deletions