Merge branch 'rs/help-unknown-ref-does-not-return'
authorJunio C Hamano <gitster@pobox.com>
Mon, 30 Sep 2019 04:19:27 +0000 (13:19 +0900)
committerJunio C Hamano <gitster@pobox.com>
Mon, 30 Sep 2019 04:19:27 +0000 (13:19 +0900)
Code cleanup.

* rs/help-unknown-ref-does-not-return:
help: make help_unknown_ref() NORETURN

1  2 
help.c
diff --combined help.c
index 5261d83ecf15042d8babccf9779f0b96fac94e92,5231c919c70bb4be553dd03a25e1066a1e4aa925..9ff2be6b188fe890e0a9ba28b63effbdf99036ef
--- 1/help.c
--- 2/help.c
+++ b/help.c
@@@ -754,19 -754,19 +754,19 @@@ static int append_similar_ref(const cha
  {
        struct similar_ref_cb *cb = (struct similar_ref_cb *)(cb_data);
        char *branch = strrchr(refname, '/') + 1;
 -      const char *remote;
  
        /* A remote branch of the same name is deemed similar */
 -      if (skip_prefix(refname, "refs/remotes/", &remote) &&
 +      if (starts_with(refname, "refs/remotes/") &&
            !strcmp(branch, cb->base_ref))
 -              string_list_append(cb->similar_refs, remote);
 +              string_list_append_nodup(cb->similar_refs,
 +                                       shorten_unambiguous_ref(refname, 1));
        return 0;
  }
  
  static struct string_list guess_refs(const char *ref)
  {
        struct similar_ref_cb ref_cb;
 -      struct string_list similar_refs = STRING_LIST_INIT_NODUP;
 +      struct string_list similar_refs = STRING_LIST_INIT_DUP;
  
        ref_cb.base_ref = ref;
        ref_cb.similar_refs = &similar_refs;
        return similar_refs;
  }
  
- void help_unknown_ref(const char *ref, const char *cmd, const char *error)
+ NORETURN void help_unknown_ref(const char *ref, const char *cmd,
+                              const char *error)
  {
        int i;
        struct string_list suggested_refs = guess_refs(ref);