FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Format code with clang-format by lucarin91 · Pull Request #35 · fastflow/fastflow · GitHub

Format code with clang-format - #35

Open
lucarin91 wants to merge 2 commits into
fastflow:masterfrom
lucarin91:format_code
Open

Format code with clang-format#35
lucarin91 wants to merge 2 commits into
fastflow:masterfrom
lucarin91:format_code

Conversation

Copy link
Copy Markdown
Contributor

I think that can be useful to have a format style for the code base, in order to be more readable and maintainable.
I propose to use clang-format because it is largely used and customizable.

I add the .clang-format that defines the used style. The style is based on the LLVM one but without the sorting of the includes to avoid compilation errors and the reflow of the comments to avoid breaking some ASCII figures in the comments. I also add some other options to inline short if and loops.

In order to run the code formatting, install clang-format (e.g., apt install clang-format) and run the script with ./scripts/format.sh to apply the style to all files.

Note: By default, the LLVM style sets the indentation width to 2 spaces, it is possible to change it to 4 spaces but I think that it makes the code more difficult to read.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL