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

Re: [PATCH v2] Remove useless uses of cat, and replace with filename arguments or redirection

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Jun 8, 2007, 12:52 UTC
Message-ID
<Pine.LNX.4.64.0706081352260.4059@racer.site>
In-Reply-To
<46a038f90706080342j3a5bb7abv58c78d7b532e29de@mail.gmail.com>
Hi,
On Fri, 8 Jun 2007, Martin Langhoff wrote:
Show 7 quoted lines
> On 6/7/07, Johannes Schindelin <Johannes.Schindelin@gmx.de> wrote:
> > Things like that are known to happen, and that's why "wc -l < file" is a
> > better fix than "wc -l file | tr -dc 0-9". In this case, it might not
> > matter for a long time, but why not stop being sloppy here and now?
> 
> Not sure if I'd call is sloppy, but I also prefer wc -l < file -- it's
> definitely safer.
Okay, so I sounded harsher than intended.

I should have phrased it like that: I consider "wc -l < file" substantially more future-proof, and it is in general a good practice IMVHO not to generate something you throw away just after that, when there is a way to avoid generating the unwanted part consistently.

Sorry, Dscho

Previous: Martin Langhoff
Message 7 of 7 in “Remove useless uses of cat, and replace with filename arguments or redirection”
  1. Remove useless uses of cat, and replace with filename arguments or redirectionJosh Triplett, Jun 6, 2007
  2. Junio C HamanoJun 7, 2007
  3. Josh TriplettJun 7, 2007
  4. Junio C HamanoJun 7, 2007
  5. Johannes SchindelinJun 7, 2007
  6. Martin LanghoffJun 8, 2007
  7. Johannes SchindelinJun 8, 2007

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.