Use c++17 in clang-format #23493

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2111-srcFormat17 changing 1 files +1 −10

MarcoFalke commented at 10:54 AM on November 12, 2021: member
We currently use Cpp11, which "is a deprecated alias for Latest" according to https://clang.llvm.org/docs/ClangFormatStyleOptions.html . I doubt this has any effect, but I think for clarity setting to c++17 make sense.

Also, remove unneeded settings:
- ObjC*, as we don't write objc code
- Penalty, as there is currently no line limit, so this has no effect
- TabWidth, as we don't use tabs
Use c++17 in clang-format faeb748f5b
MarcoFalke added the label Refactoring on Nov 12, 2021
katesalazar commented at 6:07 PM on November 13, 2021: contributor

Concept ACK.
fanquake approved
fanquake commented at 3:03 AM on November 14, 2021: member

ACK faeb748f5bd88b4de734fea40e94edbbaf8a79fd - we do have some Objc code, but it never changes.
fanquake merged this on Nov 14, 2021
fanquake closed this on Nov 14, 2021
MarcoFalke deleted the branch on Nov 14, 2021
sidhujag referenced this in commit f18d2dac4b on Nov 14, 2021
DrahtBot locked this on Nov 14, 2022

Contributors

Labels