undef/git-commit-vandalism
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge branch 'tg/stash-ref-by-index-fix'

Browse Source 
"git stash show 23" used to work, but no more after getting
rewritten in C; this regression has been corrected.

* tg/stash-ref-by-index-fix:
  stash: fix show referencing stash index
This commit is contained in:
Junio C Hamano 2019-07-09 15:25:40 -07:00
commit 99af5bedbd
2 changed files with 23 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
builtin/stash.c
View File

@ -713,11 +713,11 @@ static int git_stash_config(const char *var, const char *value, void *cb)
static int show_stash(int argc, const char **argv, const char *prefix) static int show_stash(int argc, const char **argv, const char *prefix)
{ {
int i; int i;
int opts = 0;
int ret = 0; int ret = 0;
struct stash_info info; struct stash_info info;
struct rev_info rev; struct rev_info rev;
struct argv_array stash_args = ARGV_ARRAY_INIT; struct argv_array stash_args = ARGV_ARRAY_INIT;
struct argv_array revision_args = ARGV_ARRAY_INIT;
struct option options[] = { struct option options[] = {
OPT_END() OPT_END()
}; };
@ -726,11 +726,12 @@ static int show_stash(int argc, const char **argv, const char *prefix)
git_config(git_diff_ui_config, NULL); git_config(git_diff_ui_config, NULL);
init_revisions(&rev, prefix); init_revisions(&rev, prefix);
argv_array_push(&revision_args, argv[0]);
for (i = 1; i < argc; i++) { for (i = 1; i < argc; i++) {
if (argv[i][0] != '-') if (argv[i][0] != '-')
argv_array_push(&stash_args, argv[i]); argv_array_push(&stash_args, argv[i]);
else else
opts++; argv_array_push(&revision_args, argv[i]);
} }
ret = get_stash_info(&info, stash_args.argc, stash_args.argv); ret = get_stash_info(&info, stash_args.argc, stash_args.argv);
@ -742,7 +743,7 @@ static int show_stash(int argc, const char **argv, const char *prefix)
* The config settings are applied only if there are not passed * The config settings are applied only if there are not passed
* any options. * any options.
*/ */
if (!opts) { if (revision_args.argc == 1) {
git_config(git_stash_config, NULL); git_config(git_stash_config, NULL);
if (show_stat) if (show_stat)
rev.diffopt.output_format = DIFF_FORMAT_DIFFSTAT; rev.diffopt.output_format = DIFF_FORMAT_DIFFSTAT;
@ -756,7 +757,7 @@ static int show_stash(int argc, const char **argv, const char *prefix)
} }
} }
argc = setup_revisions(argc, argv, &rev, NULL); argc = setup_revisions(revision_args.argc, revision_args.argv, &rev, NULL);
if (argc > 1) { if (argc > 1) {
free_stash_info(&info); free_stash_info(&info);
usage_with_options(git_stash_show_usage, options); usage_with_options(git_stash_show_usage, options);

18
t/t3903-stash.sh
View File

@ -708,6 +708,24 @@ test_expect_success 'invalid ref of the form "n", n >= N' '
git stash drop git stash drop
' '
test_expect_success 'valid ref of the form "n", n < N' '
git stash clear &&
echo bar5 >file &&
echo bar6 >file2 &&
git add file2 &&
git stash &&
git stash show 0 &&
git stash branch tmp 0 &&
git checkout master &&
git stash &&
git stash apply 0 &&
git reset --hard &&
git stash pop 0 &&
git stash &&
git stash drop 0 &&
test_must_fail git stash drop
'
test_expect_success 'branch: do not drop the stash if the branch exists' ' test_expect_success 'branch: do not drop the stash if the branch exists' '
git stash clear && git stash clear &&
echo foo >file && echo foo >file &&