diff options
| author |  Johannes Schindelin <johannes.schindelin@gmx.de> | 2019-09-30 02:55:31 -0700 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2019-10-04 07:35:31 +0900 |
| commit | dbcd970c27ba42ab09073211f858b1a89c7f8300 (patch) | |
| tree | 09149afddd4594195a81d8ed1d090ea730d2e9df /builtin | |
| parent | .gitignore: stop ignoring `.manifest` files (diff) | |
| download | tgif-dbcd970c27ba42ab09073211f858b1a89c7f8300.tar.xz | |
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')
| -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 021dd3b1e4..d216270d5f 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, |