diff options
author | Vicent Marti <tanoku@gmail.com> | 2011-07-05 04:28:18 +0400 |
---|---|---|
committer | Vicent Marti <tanoku@gmail.com> | 2011-07-05 04:28:18 +0400 |
commit | 9f86ec52fab7e7eeb9ee95087c08fc88185d5ff3 (patch) | |
tree | 4c59c4c8acc0c8e3001e157bee2d014c26b05575 /src/signature.c | |
parent | a01acc47bb4cc8a5c8120c95c1b054d91506ef5a (diff) |
signature: Fix warnings
Add extra braces to avoid ambiguous if-else.
Also, free() doesn't need a check.
Diffstat (limited to 'src/signature.c')
-rw-r--r-- | src/signature.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/src/signature.c b/src/signature.c index bc88a76e1..6d4860b4c 100644 --- a/src/signature.c +++ b/src/signature.c @@ -33,12 +33,8 @@ void git_signature_free(git_signature *sig) if (sig == NULL) return; - if (sig->name) - free(sig->name); - - if (sig->email) - free(sig->email); - + free(sig->name); + free(sig->email); free(sig); } @@ -66,11 +62,12 @@ static int process_trimming(const char *input, char **storage, const char *input left = skip_leading_spaces(input, input_end); right = skip_trailing_spaces(input, input_end - 1); - if (right <= left) + if (right <= left) { if (fail_when_empty) return git__throw(GIT_EINVALIDARGS, "Failed to trim. Input is either empty or only contains spaces"); else right = left - 1; + } trimmed_input_length = right - left + 1; |