diff options
author | Junio C Hamano <junkio@cox.net> | 2006-06-06 14:04:17 -0700 |
---|---|---|
committer | Junio C Hamano <junkio@cox.net> | 2006-06-06 14:30:58 -0700 |
commit | e5f38ec3c5d8553413501e6141932b8ccc7aceb4 (patch) | |
tree | 2e7b9b007e8c1dee6bbcdf14bc1ce42b4f860702 /fetch.c | |
parent | c33d5174d692b1861e0b2f656e51b80b2b00a23c (diff) | |
download | git-e5f38ec3c5d8553413501e6141932b8ccc7aceb4.tar.gz |
ref-log: style fixes.
A few style fixes to get the code in line with the rest.
- asterisk to make a type a pointer to something goes in front
of the variable, not at the end of the base type.
E.g. a pointer to an integer is "int *ip", not "int* ip".
- open parenthesis for function parameter list, unlike
syntactic constructs, comes immediately after the function
name. E.g. "if (foo) bar();" not "if(foo) bar ();".
- "else" does not come on the same line as the closing brace of
corresponding "if".
The style is mostly a matter of personal taste, and people may
disagree, but consistency is important.
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'fetch.c')
-rw-r--r-- | fetch.c | 6 |
1 files changed, 4 insertions, 2 deletions
@@ -150,7 +150,8 @@ static int process(struct object *obj) if (has_sha1_file(obj->sha1)) { /* We already have it, so we should scan it now. */ obj->flags |= TO_SCAN; - } else { + } + else { if (obj->flags & COMPLETE) return 0; prefetch(obj->sha1); @@ -255,7 +256,8 @@ int pull(char *target) if (write_ref_log_details) { msg = xmalloc(strlen(write_ref_log_details) + 12); sprintf(msg, "fetch from %s", write_ref_log_details); - } else + } + else msg = NULL; ret = write_ref_sha1(lock, sha1, msg ? msg : "fetch (unknown)"); if (msg) |