chiavdf
chiavdf copied to clipboard
Add clang-format
Dump .clang-format using:
clang-format -style=llvm -dump-config > .clang-format
and set SortIncludes to false
Run clang-format version 13.0.1-3+b1 on src/h src/.cpp src/.hpp src/.c
Rebased on main, splitted in two commits: one adding clang-format file and one executing clang-format.
'This PR has been flagged as stale due to no activity for over 60 days. It will not be automatically closed, but it has been given a stale-pr label and should be manually reviewed.'
@dannywillems I'm generally in favor of using clang format. I think the main benefit from it comes from requiring the canonical format in CI, which this patch does not introduce. This will become outdated very quickly after landing. There would probably need to be some documentation for committers how to apply the clang format before pushing a PR.
I would also have expected the clang format config to somewhat match the existing format. Most importantly, use 4 spaces for indentation rather than 2. Ideally, the change to the source code diff should be as small as possible.
'This PR has been flagged as stale due to no activity for over 60 days. It will not be automatically closed, but it has been given a stale-pr label and should be manually reviewed.'
Closing abandoned PR