From patchwork Sun Mar 24 17:51:11 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dragan Simic X-Patchwork-Id: 13600877 Received: from mail.manjaro.org (mail.manjaro.org [116.203.91.91]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3AA1418EB3 for ; Sun, 24 Mar 2024 17:51:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=116.203.91.91 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711302687; cv=none; b=SLBQvQT/Lh//cIRKa3KOMKCPv2eHPz+MxpDCThWawcxYudbtCmO/ZoEW4JJnJlH3fTzGq5GnX5d50zg4/HAVhRj1OClxI3B8woO4dIlBC681JOpdjSMMagcAUGU+s3AY1gaw93/543+2MRnO5kEV1zSYqjLfA6wzl80Vy4W9i08= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711302687; c=relaxed/simple; bh=28rinPfRAsdoVh70bz0iauUpvdwMJnfMmo+qyTwvSvM=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=Wc1DaWMfZNZhkMCjjjQdOIH6cSxIam1i1vWfDeyckO/9ysn0vX98q4YUPVt42+P/9Poj7DFcoRc1udXPQakc/3aBvJPpqNMjXSA/qJ0yS7TN7fOO2UDD344CYtVSfSABufKHPGePODlFOAPpweVBx/JE0oZ1tqijl6BPSZDWfbs= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=manjaro.org; spf=pass smtp.mailfrom=manjaro.org; dkim=pass (2048-bit key) header.d=manjaro.org header.i=@manjaro.org header.b=Nvvth7zz; arc=none smtp.client-ip=116.203.91.91 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=manjaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=manjaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=manjaro.org header.i=@manjaro.org header.b="Nvvth7zz" From: Dragan Simic DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=manjaro.org; s=2021; t=1711302676; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=dg+7xj7Ye+JzMmqAI5BCTMwiMDe08e97lMMeZWUn8sI=; b=Nvvth7zzJlm2YEttzWW9djbf8IqPTQoGZudXzCqREvz6N5T4fDry43xydfUKJTENEZDTfh Pe5SyHwFttG085kIOnzyCffaBjL6ZR2JfGTeKbEn/ydtg0jpWhx/L0hLLvJbK/7p4hmKMW JHnmzMlDm1wRLxKPjtLyH436RJdr7kPT8DDrdtSpYUWJrffUBYk5088fDN3eFszrBXi1xw Sn+8ez3+aTJYbphgAlVFPUP1EBBdI4dLsEGbESmy4u0QD9eEEFUZ5SedAOcoffXrOFMUyD z4cl7j8Od3Y51fEpmeMsuCGY4ykPttvEsN5Y7XdDwZqA165QXJB011edyEXClQ== To: git@vger.kernel.org Cc: gitster@pobox.com, sunshine@sunshineco.com, jn.avila@free.fr Subject: [PATCH v2 1/3] grep: perform some minor code and comment cleanups Date: Sun, 24 Mar 2024 18:51:11 +0100 Message-Id: <55867253d2925892f5192dbc3145f7bc9221b791.1711302588.git.dsimic@manjaro.org> In-Reply-To: References: Precedence: bulk X-Mailing-List: git@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Authentication-Results: ORIGINATING; auth=pass smtp.auth=dsimic@manjaro.org smtp.mailfrom=dsimic@manjaro.org Move some variable definitions around, and reflow one comment block, to make the code a bit neater after spotting those slightly unpolished areas. There are no functional changes to the source code. Signed-off-by: Dragan Simic --- Notes: Changes in v2: - No changes were introduced This patch is salvaged from my earlier series, [1] for which it has been concluded to be not acceptable for merging, because of possible issues with various git scripts. [2] Compared to the version in the earlies series, there are no changes in this version. As expected and as already discussed, patches like this one inevitably raise a few eyebrows. [3][4][5] [1] https://lore.kernel.org/git/cover.1710781235.git.dsimic@manjaro.org/T/#u [2] https://lore.kernel.org/git/d8475579f014a90b27efaf6207bc6fb0@manjaro.org/ [3] https://lore.kernel.org/git/CAPig+cQ6Y2oOaPkKFsD41beXLHjhD++nmf59xrcswpb6_Q-sdA@mail.gmail.com/ [4] https://lore.kernel.org/git/xmqqjzlzt61d.fsf@gitster.g/#t [5] https://lore.kernel.org/git/24093dca675c49cfde39f6d6efca2342@manjaro.org/ builtin/grep.c | 21 ++++++++------------- 1 file changed, 8 insertions(+), 13 deletions(-) diff --git a/builtin/grep.c b/builtin/grep.c index 982bcfc4b1df..af89c8b5cb19 100644 --- a/builtin/grep.c +++ b/builtin/grep.c @@ -623,13 +623,13 @@ static int grep_tree(struct grep_opt *opt, const struct pathspec *pathspec, struct tree_desc *tree, struct strbuf *base, int tn_len, int check_attr) { - struct repository *repo = opt->repo; - int hit = 0; + int hit = 0, name_base_len = 0; + int old_baselen = base->len; enum interesting match = entry_not_interesting; + struct repository *repo = opt->repo; struct name_entry entry; - int old_baselen = base->len; struct strbuf name = STRBUF_INIT; - int name_base_len = 0; + if (repo->submodule_prefix) { strbuf_addstr(&name, repo->submodule_prefix); name_base_len = name.len; @@ -890,19 +890,15 @@ static int pattern_callback(const struct option *opt, const char *arg, int cmd_grep(int argc, const char **argv, const char *prefix) { - int hit = 0; + int hit = 0, seen_dashdash = 0, use_index = 1; int cached = 0, untracked = 0, opt_exclude = -1; - int seen_dashdash = 0; int external_grep_allowed__ignored; + int i, dummy, allow_revs; const char *show_in_pager = NULL, *default_pager = "dummy"; struct grep_opt opt; struct object_array list = OBJECT_ARRAY_INIT; struct pathspec pathspec; struct string_list path_list = STRING_LIST_INIT_DUP; - int i; - int dummy; - int use_index = 1; - int allow_revs; struct option options[] = { OPT_BOOL(0, "cached", &cached, @@ -1059,9 +1055,8 @@ int cmd_grep(int argc, const char **argv, const char *prefix) recurse_submodules = 0; /* - * skip a -- separator; we know it cannot be - * separating revisions from pathnames if - * we haven't even had any patterns yet + * skip a -- separator; we know it cannot be separating revisions + * from pathnames if we haven't even had any patterns yet */ if (argc > 0 && !opt.pattern_list && !strcmp(argv[0], "--")) { argv++;