diff options
| author | Johannes Schindelin <johannes.schindelin@gmx.de> | 2019-09-30 09:55:31 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2019-10-03 22:35:31 (GMT) |
| commit | dbcd970c27ba42ab09073211f858b1a89c7f8300 (patch) | |
| tree | 09149afddd4594195a81d8ed1d090ea730d2e9df /builtin/push.c | |
| parent | aac6ff7b5beeea9bca66ecda6eec60fc1dd447ec (diff) | |
| download | git-dbcd970c27ba42ab09073211f858b1a89c7f8300.zip git-dbcd970c27ba42ab09073211f858b1a89c7f8300.tar.gz git-dbcd970c27ba42ab09073211f858b1a89c7f8300.tar.bz2 | |
push: do not pretend to return `int` from `die_push_simple()`
This function is marked as `NORETURN`, and it indeed does not want to
return anything. So let's not declare it with the return type `int`.
This fixes the following warning when building with MSVC:
C4646: function declared with 'noreturn' has non-void return type
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/push.c')
| -rw-r--r-- | builtin/push.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/builtin/push.c b/builtin/push.c index 021dd3b..d216270 100644 --- a/builtin/push.c +++ b/builtin/push.c @@ -143,8 +143,8 @@ static int push_url_of_remote(struct remote *remote, const char ***url_p) return remote->url_nr; } -static NORETURN int die_push_simple(struct branch *branch, - struct remote *remote) +static NORETURN void die_push_simple(struct branch *branch, + struct remote *remote) { /* * There's no point in using shorten_unambiguous_ref here, |