On 06.01.13 23:09, Antoine Pelisse wrote:
[snip]
> Signed-off-by: Antoine Pelisse <apeli...@gmail.com>
> ---
>  dir.c | 9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/dir.c b/dir.c
> index 9b80348..f836590 100644
> --- a/dir.c
> +++ b/dir.c
> @@ -672,7 +672,8 @@ static struct dir_entry *dir_entry_new(const char 
> *pathname, int len)
>
>  static struct dir_entry *dir_add_name(struct dir_struct *dir, const char 
> *pathname, int len)
>  {
> -     if (cache_name_exists(pathname, len, ignore_case))
> +     if (!(dir->flags & DIR_SHOW_IGNORED) &&
> +         cache_name_exists(pathname, len, ignore_case))
>               return NULL;
>
>       ALLOC_GROW(dir->entries, dir->nr+1, dir->alloc);
> @@ -877,11 +878,7 @@ static int treat_file(struct dir_struct *dir, struct 
> strbuf *path, int exclude,
>       if (exclude)
>               exclude_file = !(dir->flags & DIR_SHOW_IGNORED);
>       else if (dir->flags & DIR_SHOW_IGNORED) {
> -             /*
> -              * Optimization:
> -              * Don't spend time on indexed files, they won't be
> -              * added to the list anyway
> -              */
> +             /* Always exclude indexed files */
>               struct cache_entry *ce = index_name_exists(&the_index,
>                   path->buf, path->len, ignore_case);
>
> --
> 1.7.12.4.3.g90f5e2d
>
The bad news: the patch does not apply.
The good news: t7061 passes on pu,
and dir.c seems to be changes as needed:

commit 1f4e17c6c9833f17dc6bbf045f8a8d6378dcb417
Merge: dee1fa4 cc37e5b
Author: Junio C Hamano <gits...@pobox.com>
Date: Sun Jan 6 23:46:29 2013 -0800

Merge branch 'nd/parse-pathspec' into pu

which comes from Duy:

commit cc37e5bf18ca11d9a884bddfebcdff61df3e6279
Author: Nguyễn Thái Ngọc Duy <pclo...@gmail.com>
Date: Sun Jan 6 13:21:08 2013 +0700

Convert more init_pathspec() to parse_pathspec()



--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to