From patchwork Wed Dec 18 19:29:43 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Johannes Schindelin via GitGitGadget X-Patchwork-Id: 11301743 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id B1BF414B7 for ; Wed, 18 Dec 2019 19:29:56 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 8F87D206D7 for ; Wed, 18 Dec 2019 19:29:56 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="O5l1JcIk" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727591AbfLRT3z (ORCPT ); Wed, 18 Dec 2019 14:29:55 -0500 Received: from mail-wr1-f68.google.com ([209.85.221.68]:41560 "EHLO mail-wr1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727185AbfLRT3y (ORCPT ); Wed, 18 Dec 2019 14:29:54 -0500 Received: by mail-wr1-f68.google.com with SMTP id c9so3535135wrw.8 for ; Wed, 18 Dec 2019 11:29:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=message-id:in-reply-to:references:from:date:subject:fcc :content-transfer-encoding:mime-version:to:cc; bh=+CgvyKrM09apU1qkOsJft1P4o4wqjmDRS+ywvTmG7xU=; b=O5l1JcIknAlwSFUbXPiasoHyuf5xCOTOOkqi6bIGieIEOxeWfKfdv8oxviDfs+pbim hQkzZLA48k2bqvFh2R4+KCyIr70W2g5Nqpb/gESc45BDHDPImeJEuD5m/qexBi4voyvb QCjtrjGA+WIwi5RVUV05CGwUzFEGCqGE07LquAtGb2JiPWYQvv7yIDYAqsMfegTTr9MD Xb2u+cORtqJH3E4AtyYY6GWI/76BQNaLMuZZXZq/pgC3hscl5B9TlpmruEZd+QpFFJnl fSQ8IvXE3Qps7c6JkyUIcCRfYNJJ+VlxxS0GKSXHvxWF2lfhj3OzL47AGl1NG2ykdQQ4 vKqA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:message-id:in-reply-to:references:from:date :subject:fcc:content-transfer-encoding:mime-version:to:cc; bh=+CgvyKrM09apU1qkOsJft1P4o4wqjmDRS+ywvTmG7xU=; b=fp7FVG8S4DvpwZt2UHAGpt+bJy2L4eqrhbGxTYXVLKpLt146oqkD4Xgkfhov4YEze/ NfnuSOpFdAUEsTR32Wq0gZ4Q3miHn5yCLq3lmqGjrwemZUSvcjMQxavFzDpUnDZqOa5x Sty7J04sebdY7BNnlvFsq02lf7NbJPPDAlpHV8j+fS8xzSPFc2vx0jW6ZKGyEHL3rn9m mYU4AcgKWG+oYm28D6LyFlsG79n5kpZCy8Z2g5D1MVrdBq7GaQxIhHNJQOdMxbC+HIV6 7ecdKChV9YdlfzpR4kr2hQLne9UcyEKzlO47dpg1OvjkCmfBH5TFPbPHb5tKbs0SFBTX mhew== X-Gm-Message-State: APjAAAV4TR+hsyfrrQoKnlztjKjmI+yfzWED6Qqm5fuFtcXq6GTLerRb OBVQQK3XPjAmLbR7WR964qKy5tfl X-Google-Smtp-Source: APXvYqzqu2HOIYo0bVthV1sRQfnN8oZQnklfKXMAe7Nt9LA1pMy2ibHh65svbtkDV4j8ysu7mfw82A== X-Received: by 2002:adf:d0c1:: with SMTP id z1mr4838247wrh.371.1576697392175; Wed, 18 Dec 2019 11:29:52 -0800 (PST) Received: from [127.0.0.1] ([13.74.141.28]) by smtp.gmail.com with ESMTPSA id z18sm3421984wmf.21.2019.12.18.11.29.51 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 18 Dec 2019 11:29:51 -0800 (PST) Message-Id: In-Reply-To: References: From: "Elijah Newren via GitGitGadget" Date: Wed, 18 Dec 2019 19:29:43 +0000 Subject: [PATCH v4 5/8] dir: break part of read_directory_recursive() out for reuse Fcc: Sent MIME-Version: 1.0 To: git@vger.kernel.org Cc: blees@dcon.de, gitster@pobox.com, kyle@kyleam.com, sxlijin@gmail.com, Junio C Hamano , Elijah Newren Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org From: Elijah Newren Create an add_path_to_appropriate_result_list() function from the code at the end of read_directory_recursive() so we can use it elsewhere. Signed-off-by: Elijah Newren --- dir.c | 60 ++++++++++++++++++++++++++++++++++++----------------------- 1 file changed, 37 insertions(+), 23 deletions(-) diff --git a/dir.c b/dir.c index 517a569e10..645b44ea64 100644 --- a/dir.c +++ b/dir.c @@ -1932,6 +1932,40 @@ static void close_cached_dir(struct cached_dir *cdir) } } +static void add_path_to_appropriate_result_list(struct dir_struct *dir, + struct untracked_cache_dir *untracked, + struct cached_dir *cdir, + struct index_state *istate, + struct strbuf *path, + int baselen, + const struct pathspec *pathspec, + enum path_treatment state) +{ + /* add the path to the appropriate result list */ + switch (state) { + case path_excluded: + if (dir->flags & DIR_SHOW_IGNORED) + dir_add_name(dir, istate, path->buf, path->len); + else if ((dir->flags & DIR_SHOW_IGNORED_TOO) || + ((dir->flags & DIR_COLLECT_IGNORED) && + exclude_matches_pathspec(path->buf, path->len, + pathspec))) + dir_add_ignored(dir, istate, path->buf, path->len); + break; + + case path_untracked: + if (dir->flags & DIR_SHOW_IGNORED) + break; + dir_add_name(dir, istate, path->buf, path->len); + if (cdir->fdir) + add_untracked(untracked, path->buf + baselen); + break; + + default: + break; + } +} + /* * Read a directory tree. We currently ignore anything but * directories, regular files and symlinks. That's because git @@ -2035,29 +2069,9 @@ static enum path_treatment read_directory_recursive(struct dir_struct *dir, continue; } - /* add the path to the appropriate result list */ - switch (state) { - case path_excluded: - if (dir->flags & DIR_SHOW_IGNORED) - dir_add_name(dir, istate, path.buf, path.len); - else if ((dir->flags & DIR_SHOW_IGNORED_TOO) || - ((dir->flags & DIR_COLLECT_IGNORED) && - exclude_matches_pathspec(path.buf, path.len, - pathspec))) - dir_add_ignored(dir, istate, path.buf, path.len); - break; - - case path_untracked: - if (dir->flags & DIR_SHOW_IGNORED) - break; - dir_add_name(dir, istate, path.buf, path.len); - if (cdir.fdir) - add_untracked(untracked, path.buf + baselen); - break; - - default: - break; - } + add_path_to_appropriate_result_list(dir, untracked, &cdir, + istate, &path, baselen, + pathspec, state); } close_cached_dir(&cdir); out: