On Mon, Oct 18, 2021 at 07:09:06PM +0300, Sergey Organov wrote:
> +static int stash_staged(struct stash_info *info, const struct pathspec *ps,
> + struct strbuf *out_patch, int quiet)
> +{
> + int ret = 0;
> + struct child_process cp_diff_tree = CHILD_PROCESS_INIT;
> + struct index_state istate = { NULL };
> +
> + if (write_index_as_tree(&info->w_tree, &istate, the_repository->index_file,
> + 0, NULL)) {
> + ret = -1;
> + goto done;
> + }
> +
> + cp_diff_tree.git_cmd = 1;
> + strvec_pushl(&cp_diff_tree.args, "diff-tree", "-p", "-U1", "HEAD",
> + oid_to_hex(&info->w_tree), "--", NULL);
> + if (pipe_command(&cp_diff_tree, NULL, 0, out_patch, 0, NULL, 0)) {
> + ret = -1;
> + goto done;
> + }
> +
> + if (!out_patch->len) {
> + if (!quiet)
> + fprintf_ln(stderr, _("No staged changes"));
> + ret = 1;
> + }
> +
> +done:
> + discard_index(&istate);
> + return ret;
> +}
This function doesn't look at its "struct pathspec" parameter at all.
I'm not sure if that's a bug (i.e., it should be restricting the diff
here) or if it was just pulled unnecessarily over from stash_patch().
-Peff
