builtin-add: simplify (and increase accuracy of) exclude handling
commite96980ef8164f266308ea5fec536863a629866dc
authorJeff King <peff@peff.net>
Tue, 12 Jun 2007 21:42:14 +0000 (12 23:42 +0200)
committerJunio C Hamano <gitster@pobox.com>
Wed, 13 Jun 2007 07:41:52 +0000 (13 00:41 -0700)
treec5ad7657d86a172a6109b9f989f10c1f6c1e2648
parent2abd31b07864049f0352d86c0c90413123f40945
builtin-add: simplify (and increase accuracy of) exclude handling

Previously, the code would always set up the excludes, and then manually
pick through the pathspec we were given, assuming that non-added but
existing paths were just ignored. This was mostly correct, but would
erroneously mark a totally empty directory as 'ignored'.

Instead, we now use the collect_ignored option of dir_struct, which
unambiguously tells us whether a path was ignored. This simplifies the
code, and means empty directories are now just not mentioned at all.

Furthermore, we now conditionally ask dir_struct to respect excludes,
depending on whether the '-f' flag has been set. This means we don't have
to pick through the result, checking for an 'ignored' flag; ignored entries
were either added or not in the first place.

We can safely get rid of the special 'ignored' flags to dir_entry, which
were not used anywhere else.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Jonas Fonseca <fonseca@diku.dk>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin-add.c
dir.c
dir.h