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

Re: [PATCH] bash: offer only paths after '--'

From
Shawn O. Pearce <spearce@spearce.org>
Date
Jul 8, 2008, 23:55 UTC
Message-ID
<20080708235538.GB17263@spearce.org>
In-Reply-To
<20080708235153.GD8224@neumann>
SZEDER GGGbor <szeder@ira.uka.de> wrote:
Show 14 quoted lines
> On Tue, Jul 08, 2008 at 11:18:37PM +0000, Shawn O. Pearce wrote:
> > Junio C Hamano <gitster@pobox.com> wrote:
> > > SZEDER Gábor <szeder@ira.uka.de> writes:
> > > > +		c=$((++c))
> > > 
> > > This assignment is somewhat curious, although it should work as expected
> > > either way ;-)
>
> Well, according to
> 
>   git blame contrib/completion/git-completion.bash  |grep '++'
> 
> you started this convention back in 2006, I just copied and modified
> your code (;
Yea, I don't doubt it was me that did this.
 
> Maybe an old C++ "heritage"?  In C++ it matters for class types (e.g.
> iterators), because the postfix operator might be slower than the
> prefix.

Unlikely, but maybe. I'm not really a C++ programmer. I tend to avoid C++ when/if I am given the chance to do so.

-- 
Shawn.
Previous: SZEDER GáborNext: Junio C Hamano
Message 11 of 12 in “completion: add branch options --contains --merged --no-merged”
  1. completion: add branch options --contains --merged --no-mergedEric Raible, Jul 7, 2008
  2. Shawn O. PearceJul 8, 2008
  3. Junio C HamanoJul 8, 2008
  4. Johannes SchindelinJul 8, 2008
  5. bash: offer only paths after '--'SZEDER Gábor, Jul 8, 2008
  6. Eric RaibleJul 8, 2008
  7. Junio C HamanoJul 8, 2008
  8. Shawn O. PearceJul 8, 2008
  9. Junio C HamanoJul 8, 2008
  10. SZEDER GáborJul 8, 2008
  11. Shawn O. PearceJul 8, 2008
  12. Junio C HamanoJul 9, 2008

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.