path: root/builtin/update-index.c
diff options
authorThomas Gummerer <>2016-09-14 21:07:45 (GMT)
committerJunio C Hamano <>2016-09-14 22:03:49 (GMT)
commit22433ce4617b6ff30c9e9bf03b85d4bb244c3dec (patch)
treee7bc5d184519507b1f6fda657cf99aef1a2cb806 /builtin/update-index.c
parent840529d52c1c851f76ccc19daa88e740aac789cb (diff)
update-index: add test for chmod flags
Currently there is no test checking the expected behaviour when multiple chmod flags with different arguments are passed. As argument handling is not in line with other git commands it's easy to miss and accidentally change the current behaviour. While there, fix the argument type of chmod_path, which takes an int, but had a char passed in. Signed-off-by: Thomas Gummerer <> Signed-off-by: Junio C Hamano <>
Diffstat (limited to 'builtin/update-index.c')
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin/update-index.c b/builtin/update-index.c
index b8b8522..57bd5af 100644
--- a/builtin/update-index.c
+++ b/builtin/update-index.c
@@ -419,7 +419,7 @@ static int add_cacheinfo(unsigned int mode, const unsigned char *sha1,
return 0;
-static void chmod_path(int flip, const char *path)
+static void chmod_path(char flip, const char *path)
int pos;
struct cache_entry *ce;