public inbox for [email protected]  

help / color / mirror / Atom feed

From: Peter Eisentraut <[email protected]>
To: Daniel Gustafsson <[email protected]>
To: Bernice Southey <[email protected]>
Cc: [email protected]
Subject: Re: Indentation in a patch
Date: Wed, 26 Nov 2025 12:05:00 +0100
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <CAEDh4nxfE_dte840rgD-YM8M57qijYGv-R75MLgPRx-+=pS6hQ@mail.gmail.com>
	<[email protected]>

On 25.11.25 19:58, Daniel Gustafsson wrote:
>> On 25 Nov 2025, at 19:14, Bernice Southey <[email protected]> wrote:
> 
>> .. for my first ever patch attempt.
> 
> Awesome, we love to see new contributors!
> 
>> I'm tempted to submit my patch without indenting the
>> if-block, and an explanation.
> 
> That's perfectly fine, an incorrectly indentend diff is fine to submit if it
> makes it more readable (especially when coupled with a note explaining why).

Alternatively or additionally, git diff -w/--ignore-all-space can be 
useful in situations like this.

view thread (3+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Indentation in a patch
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox