diff options
| author | MarcoFalke <[email protected]> | 2020-06-21 05:57:48 -0400 |
|---|---|---|
| committer | MarcoFalke <[email protected]> | 2020-06-21 05:58:08 -0400 |
| commit | e6f807f87c714ac6abb8a380229a5ae50432eac9 (patch) | |
| tree | b6aed374a8fa543caea7e2f7980e0472051e55f4 /src/httpserver.cpp | |
| parent | Merge #19200: rpc: remove deprecated getaddressinfo fields (diff) | |
| parent | [tools] Update clang-format config (diff) | |
| download | discoin-e6f807f87c714ac6abb8a380229a5ae50432eac9.tar.xz discoin-e6f807f87c714ac6abb8a380229a5ae50432eac9.zip | |
Merge #19095: [tools] Update clang-format config for multi-line function declarations and calls
cc29d1e2c46e01c544ef44c79d72b7bcc5d39ba7 [tools] Update clang-format config (John Newbery)
Pull request description:
In some cases, running clang-format has made code _less_ readable by joining declarations and calls for functions with many arguments into very long lines. For example:
```
- size_t getQueueInfo(std::chrono::system_clock::time_point &first,
- std::chrono::system_clock::time_point &last) const;
+ size_t getQueueInfo(std::chrono::system_clock::time_point& first, std::chrono::system_clock::time_point& last) const;
```
(https://github.com/bitcoin/bitcoin/pull/19090#discussion_r431961148)
This change to clang-format would allow arguments/parameters for func declarations/calls to be split over multiple lines, aligned with the opening parens. It does not force args/params to be on new lines (that setting is `BinPackParameters : true`).
ACKs for top commit:
MarcoFalke:
ACK cc29d1e2c46e01c544ef44c79d72b7bcc5d39ba7 fine with me
practicalswift:
ACK cc29d1e2c46e01c544ef44c79d72b7bcc5d39ba7
Tree-SHA512: a62474925e71aaff41bdce7960fd5ffd64317da810f694d8084080b054708cf71c2ab2ce3111db5a9260d1c1f9e02d59a2ecb5543b1b6172ce085cb42432160a
Diffstat (limited to 'src/httpserver.cpp')
0 files changed, 0 insertions, 0 deletions