[PATCH v2 4/8] mv: remove an "if" that's always true
- From
Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
- Date
- Aug 10, 2014, 02:29 UTC
- Message-ID
- <1407637776-19794-5-git-send-email-pclouds@gmail.com>
- In-Reply-To
- <1407637776-19794-1-git-send-email-pclouds@gmail.com>
This is inside an "else" block of "if (last - first < 1)", so we know that "last - first >= 1" when we come here. No need to check "last - first > 0".
While at there, save "argc + last - first" to a variable to shorten the statements a bit.
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com> --- builtin/mv.c | 22 +++++++--------------- 1 file changed, 7 insertions(+), 15 deletions(-)
diff --git a/builtin/mv.c b/builtin/mv.c index 524f4e5..0732355 100644 --- a/builtin/mv.c +++ b/builtin/mv.c @@ -175,22 +175,14 @@ int cmd_mv(int argc, const char **argv, const char *prefix) if (last - first < 1) bad = _("source directory is empty"); else { - int j, dst_len; + int j, dst_len, n; - if (last - first > 0) { - source = xrealloc(source, - (argc + last - first) - * sizeof(char *)); - destination = xrealloc(destination, - (argc + last - first) - * sizeof(char *)); - modes = xrealloc(modes, - (argc + last - first) - * sizeof(enum update_mode)); - submodule_gitfile = xrealloc(submodule_gitfile, - (argc + last - first) - * sizeof(char *)); - } + n = argc + last - first; + source = xrealloc(source, n * sizeof(char *)); + destination = xrealloc(destination, n * sizeof(char *)); + modes = xrealloc(modes, n * sizeof(enum update_mode)); + submodule_gitfile = + xrealloc(submodule_gitfile, n * sizeof(char *)); dst = add_slash(dst); dst_len = strlen(dst);
-- 2.1.0.rc0.78.gc0d8480