Thread (126 messages) flat view 126 messages, 3 authors, 2017-01-09

Re: [PATCH v4 00/16] pathspec cleanup

From: Brandon Williams <hidden>
Date: 2017-01-04 17:56:24

On 01/04, Brandon Williams wrote:
On 01/04, Duy Nguyen wrote:
quoted
On Wed, Jan 4, 2017 at 1:42 AM, Brandon Williams [off-list ref] wrote:
quoted
diff --git a/dir.c b/dir.c
index 15f7c9993..e8ddd7f8a 100644
--- a/dir.c
+++ b/dir.c
@@ -1353,6 +1353,15 @@ static int simplify_away(const char *path, int pathlen,
 {
        int i;

+       if (pathspec)
+               guard_pathspec(pathspec,
+                              pathspec_fromtop |
+                              pathspec_maxdepth |
+                              pathspec_literal |
+                              pathspec_glob |
+                              pathspec_icase |
+                              pathspec_exclude);
You have done some magic (or your MTA/editor did) to lower case
GUARD_PATHSPEC and all the flags. The real patch looks good though, so
no problem.
That's really odd, I was sure I just did a cut and paste.  I'll fix that
:)
So it looks like what ever I did to insert this into the cover letter
made everything lower case.  The actual patch itself looks fine.

-- 
Brandon Williams
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help