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

Re: [PATCH 09/17] pathspec: always show mnemonic and name in unsupported_magic

From: Duy Nguyen <hidden>
Date: 2016-12-07 10:48:31

On Wed, Dec 7, 2016 at 4:51 AM, Brandon Williams [off-list ref] wrote:
quoted hunk ↗ jump to hunk
@@ -426,8 +423,7 @@ void parse_pathspec(struct pathspec *pathspec,
                        nr_exclude++;
                if (item[i].magic & magic_mask)
                        unsupported_magic(entry,
-                                         item[i].magic & magic_mask,
-                                         short_magic);
+                                         item[i].magic & magic_mask);
Same here. Maybe put both arguments in the same line. It looks a bit
better. (sorry for two mails on the same patch, I'm reading the final
output first before going through individual patches that breaks this
function down)
                if ((flags & PATHSPEC_SYMLINK_LEADING_PATH) &&
                    has_symlink_leading_path(item[i].match, item[i].len)) {
-- 
Duy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help