Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code formatting: Change indentation style to space #379

Closed
olofhagsand opened this issue Oct 7, 2022 · 1 comment
Closed

Code formatting: Change indentation style to space #379

olofhagsand opened this issue Oct 7, 2022 · 1 comment

Comments

@olofhagsand
Copy link
Member

In clixon source, there is an .editorconfig as follows

indent_style = tab
indent_size  = 4

Currently there are TABS inserted in places that are not completely logical.
It is proposed to remove the use of TABS completely, ie always use spaces instead of TABs in C-code.
In emacs, this means: (setq-default indent-tabs-mode nil)
A small sample of other C-based projects seems to indicate that this seems common practice.
It is therefore also proposed to update editorconfig to (at least):

indent_style = space
indent_size  = 4

This will result in a major code change (ie only appearance), replacing all tabs to spaces.
See also
#373

olofhagsand added a commit to clicon/cligen that referenced this issue Oct 27, 2022
  * Applies to all c/h/y/l/sh files and .editorconfig
olofhagsand added a commit that referenced this issue Oct 27, 2022
  * Applies to all c/h/y/l/sh files and .editorconfig
@olofhagsand
Copy link
Member Author

All cligen and clixon code changed from TAB formatting to SPACE.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant