Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH 1/2] docs: add a basic description of the config API

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:53:01

Jeff King [off-list ref] writes:
quoted hunk
-- >8 --
Subject: drop odd return value semantics from userdiff_config
...
 5 files changed, 12 insertions(+), 37 deletions(-)
diff --git a/builtin/blame.c b/builtin/blame.c
index 5a67c20..01956c8 100644
--- a/builtin/blame.c
+++ b/builtin/blame.c
@@ -2050,14 +2050,8 @@ static int git_blame_config(const char *var, const char *value, void *cb)
 		return 0;
 	}
 
-	switch (userdiff_config(var, value)) {
-	case 0:
-		break;
-	case -1:
+	if (userdiff_config(var, value) < 0)
 		return -1;
-	default:
-		return 0;
-	}
 
 	return git_default_config(var, value, cb);
 }
...
Looks very nice ;-)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help