Re: [PATCH 0/5] fc/transport-helper-sync-error-fix rebased

From: Felipe Contreras <felipe.contreras@gmail.com>
Date: Apr 20, 2014, 21:54 UTC
Message-ID: <53544213e6635_a81169b2ece1@nysa.notmuch>
In-Reply-To: <7v7g6jllck.fsf@alter.siamese.dyndns.org>

Junio C Hamano wrote:

Show 5 quoted lines

> So I think it is better to quote these in this particular case.
> 
> The pipe is purely subjective readability. I can go either way, but
> since I was the one who was applying the patch that needed other
> changes anyway... ;-)

You started by saying these were "not the matter for your liking or not liking", and now you are explaining why you like one better than the other.

I don't think it's particularly helpful to criticize one person's preferences because they don't match yours.

You like one, I like another. That's it.

-- 
Felipe Contreras

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