style: the opening '{' of a function is in a separate line

pclouds · gitster · commit 3b3357626edc · 2018-12-10T15:41:09.000+09:00
Signed-off-by: Nguyễn Thái Ngọc Duy &lt;pclouds@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/bisect.c b/bisect.c
@@ -558,7 +558,8 @@ struct commit_list *filter_skipped(struct commit_list *list,
  * is increased by one between each call, but that should not matter
  * for this application.
  */
-static unsigned get_prn(unsigned count) {
+static unsigned get_prn(unsigned count)
+{
 	count = count * 1103515245 + 12345;
 	return (count/65536) % PRN_MODULO;
 }
diff --git a/builtin/config.c b/builtin/config.c
@@ -164,7 +164,8 @@ static NORETURN void usage_builtin_config(void)
 	usage_with_options(builtin_config_usage, builtin_config_options);
 }
 
-static void check_argc(int argc, int min, int max) {
+static void check_argc(int argc, int min, int max)
+{
 	if (argc >= min && argc <= max)
 		return;
 	if (min == max)
diff --git a/builtin/push.c b/builtin/push.c
@@ -143,7 +143,9 @@ static int push_url_of_remote(struct remote *remote, const char ***url_p)
 	return remote->url_nr;
 }
 
-static NORETURN int die_push_simple(struct branch *branch, struct remote *remote) {
+static NORETURN int die_push_simple(struct branch *branch,
+				    struct remote *remote)
+{
 	/*
 	 * There's no point in using shorten_unambiguous_ref here,
 	 * as the ambiguity would be on the remote side, not what
diff --git a/convert.c b/convert.c
@@ -778,7 +778,8 @@ static int start_multi_file_filter_fn(struct subprocess_entry *subprocess)
 
 static void handle_filter_error(const struct strbuf *filter_status,
 				struct cmd2process *entry,
-				const unsigned int wanted_capability) {
+				const unsigned int wanted_capability)
+{
 	if (!strcmp(filter_status->buf, "error"))
 		; /* The filter signaled a problem with the file. */
 	else if (!strcmp(filter_status->buf, "abort") && wanted_capability) {
diff --git a/credential-cache--daemon.c b/credential-cache--daemon.c
@@ -91,7 +91,8 @@ static timestamp_t check_expirations(void)
 }
 
 static int read_request(FILE *fh, struct credential *c,
-			struct strbuf *action, int *timeout) {
+			struct strbuf *action, int *timeout)
+{
 	static struct strbuf item = STRBUF_INIT;
 	const char *p;
 
diff --git a/diff.c b/diff.c
@@ -1637,7 +1637,8 @@ static void emit_hunk_header(struct emit_callback *ecbdata,
 	strbuf_release(&msgbuf);
 }
 
-static struct diff_tempfile *claim_diff_tempfile(void) {
+static struct diff_tempfile *claim_diff_tempfile(void)
+{
 	int i;
 	for (i = 0; i < ARRAY_SIZE(diff_temp); i++)
 		if (!diff_temp[i].name)
@@ -4819,7 +4820,8 @@ static int parse_diff_filter_opt(const char *optarg, struct diff_options *opt)
 	return 0;
 }
 
-static void enable_patch_output(int *fmt) {
+static void enable_patch_output(int *fmt)
+{
 	*fmt &= ~DIFF_FORMAT_NO_OUTPUT;
 	*fmt |= DIFF_FORMAT_PATCH;
 }
diff --git a/git.c b/git.c
@@ -98,7 +98,8 @@ static int list_cmds(const char *spec)
 	return 0;
 }
 
-static void commit_pager_choice(void) {
+static void commit_pager_choice(void)
+{
 	switch (use_pager) {
 	case 0:
 		setenv("GIT_PAGER", "cat", 1);
diff --git a/imap-send.c b/imap-send.c
@@ -1471,7 +1471,8 @@ static CURL *setup_curl(struct imap_server_conf *srvc, struct credential *cred)
 }
 
 static int curl_append_msgs_to_imap(struct imap_server_conf *server,
-				    struct strbuf* all_msgs, int total) {
+				    struct strbuf* all_msgs, int total)
+{
 	int ofs = 0;
 	int n = 0;
 	struct buffer msgbuf = { STRBUF_INIT, 0 };
diff --git a/remote-curl.c b/remote-curl.c
@@ -617,7 +617,8 @@ static int probe_rpc(struct rpc_state *rpc, struct slot_results *results)
 	return err;
 }
 
-static curl_off_t xcurl_off_t(size_t len) {
+static curl_off_t xcurl_off_t(size_t len)
+{
 	uintmax_t size = len;
 	if (size > maximum_signed_value_of_type(curl_off_t))
 		die("cannot handle pushes this big");
diff --git a/sequencer.c b/sequencer.c
@@ -1670,7 +1670,8 @@ static int update_squash_messages(enum todo_command command,
 	return res;
 }
 
-static void flush_rewritten_pending(void) {
+static void flush_rewritten_pending(void)
+{
 	struct strbuf buf = STRBUF_INIT;
 	struct object_id newoid;
 	FILE *out;
@@ -1695,7 +1696,8 @@ static void flush_rewritten_pending(void) {
 }
 
 static void record_in_rewritten(struct object_id *oid,
-		enum todo_command next_command) {
+		enum todo_command next_command)
+{
 	FILE *out = fopen_or_warn(rebase_path_rewritten_pending(), "a");
 
 	if (!out)
diff --git a/string-list.c b/string-list.c
@@ -155,7 +155,8 @@ static int item_is_not_empty(struct string_list_item *item, void *unused)
 	return *item->string != '\0';
 }
 
-void string_list_remove_empty_items(struct string_list *list, int free_util) {
+void string_list_remove_empty_items(struct string_list *list, int free_util)
+{
 	filter_string_list(list, free_util, item_is_not_empty, NULL);
 }
 
diff --git a/t/helper/test-sigchain.c b/t/helper/test-sigchain.c
@@ -14,7 +14,8 @@ X(two)
 X(three)
 #undef X
 
-int cmd__sigchain(int argc, const char **argv) {
+int cmd__sigchain(int argc, const char **argv)
+{
 	sigchain_push(SIGTERM, one);
 	sigchain_push(SIGTERM, two);
 	sigchain_push(SIGTERM, three);
diff --git a/transport-helper.c b/transport-helper.c
@@ -1026,7 +1026,8 @@ static int push_refs(struct transport *transport,
 }
 
 
-static int has_attribute(const char *attrs, const char *attr) {
+static int has_attribute(const char *attrs, const char *attr)
+{
 	int len;
 	if (!attrs)
 		return 0;
diff --git a/url.c b/url.c
@@ -104,7 +104,8 @@ void end_url_with_slash(struct strbuf *buf, const char *url)
 	strbuf_complete(buf, '/');
 }
 
-void str_end_url_with_slash(const char *url, char **dest) {
+void str_end_url_with_slash(const char *url, char **dest)
+{
 	struct strbuf buf = STRBUF_INIT;
 	end_url_with_slash(&buf, url);
 	free(*dest);
diff --git a/userdiff.c b/userdiff.c
@@ -265,7 +265,8 @@ int userdiff_config(const char *k, const char *v)
 	return 0;
 }
 
-struct userdiff_driver *userdiff_find_by_name(const char *name) {
+struct userdiff_driver *userdiff_find_by_name(const char *name)
+{
 	int len = strlen(name);
 	return userdiff_find_by_namelen(name, len);
 }

Original file line number	Diff line number	Diff line change
`@@ -164,7 +164,8 @@ static NORETURN void usage_builtin_config(void)`
`164`	`164`	`usage_with_options(builtin_config_usage, builtin_config_options);`
`165`	`165`	`}`
`166`	`166`
`167`		`-static void check_argc(int argc, int min, int max) {`
	`167`	`+static void check_argc(int argc, int min, int max)`
	`168`	`+{`
`168`	`169`	`if (argc >= min && argc <= max)`
`169`	`170`	`return;`
`170`	`171`	`if (min == max)`
Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,8 @@ static timestamp_t check_expirations(void)`
`91`	`91`	`}`
`92`	`92`
`93`	`93`	`static int read_request(FILE fh, struct credential c,`
`94`		`- struct strbuf action, int timeout) {`
	`94`	`+ struct strbuf action, int timeout)`
	`95`	`+{`
`95`	`96`	`static struct strbuf item = STRBUF_INIT;`
`96`	`97`	`const char *p;`
`97`	`98`
Original file line number	Diff line number	Diff line change
`@@ -1637,7 +1637,8 @@ static void emit_hunk_header(struct emit_callback *ecbdata,`
`1637`	`1637`	`strbuf_release(&msgbuf);`
`1638`	`1638`	`}`
`1639`	`1639`
`1640`		`-static struct diff_tempfile *claim_diff_tempfile(void) {`
	`1640`	`+static struct diff_tempfile *claim_diff_tempfile(void)`
	`1641`	`+{`
`1641`	`1642`	`int i;`
`1642`	`1643`	`for (i = 0; i < ARRAY_SIZE(diff_temp); i++)`
`1643`	`1644`	`if (!diff_temp[i].name)`
`@@ -4819,7 +4820,8 @@ static int parse_diff_filter_opt(const char optarg, struct diff_options opt)`
`4819`	`4820`	`return 0;`
`4820`	`4821`	`}`
`4821`	`4822`
`4822`		`-static void enable_patch_output(int *fmt) {`
	`4823`	`+static void enable_patch_output(int *fmt)`
	`4824`	`+{`
`4823`	`4825`	`*fmt &= ~DIFF_FORMAT_NO_OUTPUT;`
`4824`	`4826`	`*fmt \|= DIFF_FORMAT_PATCH;`
`4825`	`4827`	`}`
Original file line number	Diff line number	Diff line change
`@@ -98,7 +98,8 @@ static int list_cmds(const char *spec)`
`98`	`98`	`return 0;`
`99`	`99`	`}`
`100`	`100`
`101`		`-static void commit_pager_choice(void) {`
	`101`	`+static void commit_pager_choice(void)`
	`102`	`+{`
`102`	`103`	`switch (use_pager) {`
`103`	`104`	`case 0:`
`104`	`105`	`setenv("GIT_PAGER", "cat", 1);`
Original file line number	Diff line number	Diff line change
`@@ -1471,7 +1471,8 @@ static CURL setup_curl(struct imap_server_conf srvc, struct credential *cred)`
`1471`	`1471`	`}`
`1472`	`1472`
`1473`	`1473`	`static int curl_append_msgs_to_imap(struct imap_server_conf *server,`
`1474`		`- struct strbuf* all_msgs, int total) {`
	`1474`	`+ struct strbuf* all_msgs, int total)`
	`1475`	`+{`
`1475`	`1476`	`int ofs = 0;`
`1476`	`1477`	`int n = 0;`
`1477`	`1478`	`struct buffer msgbuf = { STRBUF_INIT, 0 };`
Original file line number	Diff line number	Diff line change
`@@ -617,7 +617,8 @@ static int probe_rpc(struct rpc_state rpc, struct slot_results results)`
`617`	`617`	`return err;`
`618`	`618`	`}`
`619`	`619`
`620`		`-static curl_off_t xcurl_off_t(size_t len) {`
	`620`	`+static curl_off_t xcurl_off_t(size_t len)`
	`621`	`+{`
`621`	`622`	`uintmax_t size = len;`
`622`	`623`	`if (size > maximum_signed_value_of_type(curl_off_t))`
`623`	`624`	`die("cannot handle pushes this big");`