git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] grep: load funcname patterns for -W

From
René Scharfe <rene.scharfe@lsrfire.ath.cx>
Date
Nov 25, 2011, 16:32 UTC
Message-ID
<4ECFC320.4030003@lsrfire.ath.cx>
In-Reply-To
<5e3bcf651b31b299ca411296e6e7c4d11f6ae617.1322232319.git.trast@student.ethz.ch>
Am 25.11.2011 15:46, schrieb Thomas Rast:
Show 8 quoted lines
> git-grep avoids loading the funcname patterns unless they are needed.
> ba8ea74 (grep: add option to show whole function as context,
> 2011-08-01) forgot to extend this test also to the new funcbody
> feature.  Do so.
> 
> The catch is that we also have to disable threading when using
> userdiff, as explained in grep_threads_ok().  So we must be careful to
> introduce the same test there.
Oops.  Thanks for catching this.

That reminds me to look into adding a thread-safe way to access attributes again..

Thanks, René

Previous: Thomas RastNext: René Scharfe
Message 2 of 6 in “grep: load funcname patterns for -W”
  1. grep: load funcname patterns for -WThomas Rast, Nov 25, 2011
  2. René ScharfeNov 25, 2011
  3. grep: enable multi-threading for -p and -WRené Scharfe, Nov 26, 2011
  4. Thomas RastNov 29, 2011
  5. René ScharfeNov 29, 2011
  6. Thomas RastNov 29, 2011

Read the whole thread, see it on lore, or plain text.

$ 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.