From: shyouhei@...
Date: 2018-01-15T02:26:27+00:00
Subject: [ruby-core:84863] [Ruby trunk Bug#14246] Inconsistent C source code	indentation

Issue #14246 has been updated by shyouhei (Shyouhei Urabe).


@graywolf we are talking about C codes here.  The "Do not use TABs" policy applies to non-C files.

----------------------------------------
Bug #14246: Inconsistent C source code indentation
https://bugs.ruby-lang.org/issues/14246#change-69577

* Author: shyouhei (Shyouhei Urabe)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
* ruby -v: 
* Backport: 2.3: UNKNOWN, 2.4: UNKNOWN, 2.5: UNKNOWN
----------------------------------------
I would like to focus on following 238 C source codes in our repository.

```
% git ls-files | grep '\.\(c\|h\|def\)$' | grep -v 'ext|spec|test' | wc -l
238
```

Here, in these 238 files, 10 files are indented using spaces only.

```
% git ls-files | grep '\.\(c\|h\|def\)$' | grep -v 'ext|spec|test' | \
xargs grep --files-without-match $'^\t' | xargs grep --files-with-match '^        ' | wc -l
10
```

On the other hand 66 files are indented using tabs.

```
% git ls-files | grep '\.\(c\|h\|def\)$' | grep -v 'ext|spec|test' | \
xargs grep --files-with-match $'^\t' | xargs grep --files-without-match '^        ' | wc -l
66
```

Other 61 files do not indent.  We should not bother them.

```
% git ls-files | grep '\.\(c\|h\|def\)$' | grep -v 'ext|spec|test' | \
xargs grep --files-without-match $'^\t' | xargs grep --files-without-match '^        ' | wc -l
61
```

So far so good.  But what about remaining 101 files?  The answer is obvious; these files MIX indents.

```
% git ls-files | grep '\.\(c\|h\|def\)$' | grep -v 'ext|spec|test' | \
xargs grep --files-with-match $'^\t' | xargs grep --files-with-match '^        ' | wc -l
101
```

This is totally wrong.  No matter should we use spaces or tabs for indentations, it must be consistent.



-- 
https://bugs.ruby-lang.org/

Unsubscribe: <mailto:ruby-core-request@ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>