From: Junio C Hamano <gitster@pobox.com>
Date: Sun, 24 Sep 2017 10:39:39 +0000 (+0900)
Subject: Merge branch 'jk/describe-omit-some-refs' into next
X-Git-Url: https://git.lorimer.id.au/gitweb.git/diff_plain/c373c7127984764c36e72738ad2ea6b14459a631?hp=--cc

Merge branch 'jk/describe-omit-some-refs' into next

"git describe --match" learned to take multiple patterns in v2.13
series, but the feature ignored the patterns after the first one
and did not work at all.  This has been fixed.

* jk/describe-omit-some-refs:
describe: fix matching to actually match all patterns
---

c373c7127984764c36e72738ad2ea6b14459a631
diff --cc builtin/describe.c
index e77163e909,ba4ebb4d1b..3dc1836480
--- a/builtin/describe.c
+++ b/builtin/describe.c
@@@ -164,12 -158,14 +165,14 @@@ static int get_name(const char *path, c
  			return 0;
  
  		for_each_string_list_item(item, &patterns) {
- 			if (!wildmatch(item->string, path + 10, 0))
 -			if (!wildmatch(item->string, path + 10, 0, NULL)) {
++			if (!wildmatch(item->string, path + 10, 0)) {
+ 				found = 1;
  				break;
+ 			}
+ 		}
  
- 			/* If we get here, no pattern matched. */
+ 		if (!found)
  			return 0;
- 		}
  	}
  
  	/* Is it annotated? */