Re: [PATCH v2 00/12] nd/icase updates

From: Junio C Hamano <gitster@pobox.com>
Date: Jun 27, 2016, 15:00 UTC
Message-ID: <xmqqa8i6d6dl.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <xmqqeg7id6ns.fsf@gitster.mtv.corp.google.com>

Junio C Hamano <gitster@pobox.com> writes:

Show 11 quoted lines

>  	if (p->fixed) {
>  		p->kws = kwsalloc(icase ? tolower_trans_tbl : NULL);
>  		kwsincr(p->kws, p->pattern, p->patternlen);
>  		kwsprep(p->kws);
>  		return;
>  	} else if (opt->fixed) {
> +		/*
> +		 * We only come here when the pattern has the regexp
> +		 * special characters in it, which need to be matched
> +		 * literally, while ignoring case.
> +		 */

This comment is wrong.

		/*
		 * We come here only when the pattern has non-ascii
		 * characters we cannot case-fold, and we were asked
		 * to ignore-case.
		 */

>  		compile_fixed_regexp(p, opt);
>  		return;
>  	}

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.