summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: 971dfa1)
raw | patch | inline | side by side (parent: 971dfa1)
author | Junio C Hamano <gitster@pobox.com> | |
Fri, 7 Mar 2008 07:31:29 +0000 (23:31 -0800) | ||
committer | Junio C Hamano <gitster@pobox.com> | |
Fri, 7 Mar 2008 08:14:43 +0000 (00:14 -0800) |
When get_pathspec() was originally made absolute-path capable,
we botched the interface to it, without dying inside the function
when given a path that is outside the work tree, and made it the
responsibility of callers to check the condition in a roundabout
way. This is made unnecessary with the previous patch.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
we botched the interface to it, without dying inside the function
when given a path that is outside the work tree, and made it the
responsibility of callers to check the condition in a roundabout
way. This is made unnecessary with the previous patch.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin-add.c | patch | blob | history |
diff --git a/builtin-add.c b/builtin-add.c
index 820110e085f20d8615c74167cf3ae4cfd1fe6912..4a91e3eb118850882fbc22e8d8f37e8bbfaa7617 100644 (file)
--- a/builtin-add.c
+++ b/builtin-add.c
goto finish;
}
- if (*argv) {
- /* Was there an invalid path? */
- if (pathspec) {
- int num;
- for (num = 0; pathspec[num]; num++)
- ; /* just counting */
- if (argc != num)
- exit(1); /* error message already given */
- } else
- exit(1); /* error message already given */
- }
-
fill_directory(&dir, pathspec, ignored_too);
if (show_only) {