diff options
| author | Junio C Hamano <gitster@pobox.com> | 2013-10-23 13:21:35 -0700 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2013-10-23 13:21:35 -0700 |
| commit | f2c1b01c24b6c1df962a6526661cf3a58f3c6a56 (patch) | |
| tree | d77d3cf65e96930095dbc40f76cb356287696258 /builtin/revert.c | |
| parent | 4197361e39a304df30cc492122dbdfe90ae8af0e (diff) | |
| parent | d644c5502fc576904a07fab35f43540fa9f2e7c2 (diff) | |
| download | git-f2c1b01c24b6c1df962a6526661cf3a58f3c6a56.tar.gz git-f2c1b01c24b6c1df962a6526661cf3a58f3c6a56.tar.xz | |
Merge branch 'hu/cherry-pick-previous-branch'
"git cherry-pick" without further options would segfault.
Could use a follow-up to handle '-' after argv[1] better.
* hu/cherry-pick-previous-branch:
cherry-pick: handle "-" after parsing options
Diffstat (limited to 'builtin/revert.c')
| -rw-r--r-- | builtin/revert.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/builtin/revert.c b/builtin/revert.c index 52c35e75d..87659c9fd 100644 --- a/builtin/revert.c +++ b/builtin/revert.c @@ -168,6 +168,8 @@ static void parse_args(int argc, const char **argv, struct replay_opts *opts) opts->revs->no_walk = REVISION_WALK_NO_WALK_UNSORTED; if (argc < 2) usage_with_options(usage_str, options); + if (!strcmp(argv[1], "-")) + argv[1] = "@{-1}"; memset(&s_r_opt, 0, sizeof(s_r_opt)); s_r_opt.assume_dashdash = 1; argc = setup_revisions(argc, argv, opts->revs, &s_r_opt); @@ -202,8 +204,6 @@ int cmd_cherry_pick(int argc, const char **argv, const char *prefix) memset(&opts, 0, sizeof(opts)); opts.action = REPLAY_PICK; git_config(git_default_config, NULL); - if (!strcmp(argv[1], "-")) - argv[1] = "@{-1}"; parse_args(argc, argv, &opts); res = sequencer_pick_revisions(&opts); if (res < 0) |