[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/4] Indent maybe_os stuff ahead of refactor
From: |
Ben Elliston |
Subject: |
Re: [PATCH 2/4] Indent maybe_os stuff ahead of refactor |
Date: |
Thu, 3 May 2018 04:52:49 +1000 |
User-agent: |
NeoMutt/20170609 (1.8.3) |
On Tue, May 01, 2018 at 04:12:18PM -0400, John Ericson wrote:
> Sure, but I'm a bit confused on what you intended. There's a mix of
> tabs and spaces which leads me to think you are using 4 space indent
> with a tab for every 8 spaces? But the rest of the file looks like
> it is using a tab per indent level (such that the spaces/tab doesn't
> matter), where 4 spaces/tab is intended so the indent visually
> matches what you did.
Just mimic what is already there. What's there doesn't match what my
Emacs shell-scipt-mode wants to with formatting, but I am not inclined
to disturb the file unecessarily. It's been fine this way for 25
years. :-)
Thanks, Ben
signature.asc
Description: PGP signature
- Re: [PATCH 2/4] Indent maybe_os stuff ahead of refactor, John Ericson, 2018/05/01
- Re: [PATCH 2/4] Indent maybe_os stuff ahead of refactor,
Ben Elliston <=
- Re: [PATCH 2/4] Indent maybe_os stuff ahead of refactor, John Ericson, 2018/05/02
- [PATCH 1/2] * config.sub: Properly recognise configs with 4 components, John Ericson, 2018/05/02
- [PATCH 2/2] * config.sub (arm*-*-none-eabi): Recognise, John Ericson, 2018/05/02
- Re: [PATCH 1/2] * config.sub: Properly recognise configs with 4 components, Ben Elliston, 2018/05/04
- Re: [PATCH 1/2] * config.sub: Properly recognise configs with 4 components, John Ericson, 2018/05/04
- [PATCH 1/2] * config.sub: Properly recognise configs with 4 components, John Ericson, 2018/05/04
- [PATCH 2/2] * config.sub (arm*-*-none-eabi): Recognise, John Ericson, 2018/05/04
- Re: [PATCH 2/2] * config.sub (arm*-*-none-eabi): Recognise, Ben Elliston, 2018/05/05
- Re: [PATCH 2/2] * config.sub (arm*-*-none-eabi): Recognise, John Ericson, 2018/05/11
- Re: [PATCH 2/2] * config.sub (arm*-*-none-eabi): Recognise, Ben Elliston, 2018/05/11