Fix two issues:
- Remove unneeded code, which makes test harder to read and write
- Fix wrong indentation, which breaks some editors
test: tidy up addrman unit tests #23492
pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2111-testAddrm changing 1 files +23 −28-
MarcoFalke commented at 10:43 AM on November 12, 2021: member
-
test: Remove unused CDataStream copy fab9264be5
-
0000edaba3
style: Use 4 spaces for indentation, not 5
The wrong indentation breaks editor workflows. Can be reviewed with --ignore-all-space
- fanquake added the label Tests on Nov 12, 2021
- brunoerg approved
-
brunoerg commented at 11:18 AM on November 12, 2021: member
cr/tACK 0000edaba306487a750f5648438f3dc8de08127c
- shaavan approved
-
shaavan commented at 12:32 PM on November 12, 2021: contributor
ACK 0000edaba306487a750f5648438f3dc8de08127c
-
jnewbery commented at 12:39 PM on November 12, 2021: member
Code review ACK 0000edaba306487a750f5648438f3dc8de08127c
- MarcoFalke referenced this in commit c9dd5c8d6e on Nov 12, 2021
- MarcoFalke closed this on Nov 12, 2021
- MarcoFalke deleted the branch on Nov 12, 2021
- MarcoFalke restored the branch on Nov 12, 2021
- MarcoFalke deleted the branch on Nov 12, 2021
- MarcoFalke restored the branch on Nov 12, 2021
- MarcoFalke deleted the branch on Nov 12, 2021
- sidhujag referenced this in commit cf03e2cb44 on Nov 12, 2021
- DrahtBot locked this on Nov 12, 2022
Contributors
Labels