Double semicolon cleanup #7263

pull ghost wants to merge 1 commits into bitcoin:master from changing 6 files +9 −9

ghost commented at 3:45 AM on December 30, 2015: none

Assuming, and clarifying, the empty statements weren't intentional.

paveljanik commented at 7:41 AM on December 30, 2015: contributor

Can you please remove also the rest?

qt/bantablemodel.cpp:    return columns.length();;
qt/guiutil.cpp:        QString msg(QUrl::toPercentEncoding(info.message));;
qt/peertablemodel.cpp:    return columns.length();;
test/rpc_tests.cpp:    BOOST_CHECK_NO_THROW(CallRPC(string("setban 127.0.0.0 remove")));;
test/rpc_tests.cpp:    BOOST_CHECK_NO_THROW(CallRPC(string("setban 127.0.0.0/24 remove")));;

Double semicolon cleanup. a5a0831458
ghost commented at 5:25 PM on December 30, 2015: none

PR updated accordingly. Why don't we spin off Qt!?
MarcoFalke commented at 5:43 PM on December 30, 2015: member

utACK a5a0831458d8290c1e7591cf32a529669b613d86
paveljanik commented at 6:01 PM on December 30, 2015: contributor

ACK https://github.com/bitcoin/bitcoin/commit/a5a0831458d8290c1e7591cf32a529669b613d86
fanquake commented at 3:18 AM on December 31, 2015: member

ACK https://github.com/21E14/bitcoin/commit/a5a0831458d8290c1e7591cf32a529669b613d86
jonasschnelli commented at 7:20 AM on January 2, 2016: contributor

ACK a5a0831458d8290c1e7591cf32a529669b613d86
jonasschnelli added the label Refactoring on Jan 2, 2016
paveljanik commented at 1:18 PM on January 3, 2016: contributor

ACK https://github.com/bitcoin/bitcoin/commit/a5a0831458d8290c1e7591cf32a529669b613d86
laanwj merged this on Jan 4, 2016
laanwj closed this on Jan 4, 2016
laanwj referenced this in commit ac982a16e0 on Jan 4, 2016
codablock referenced this in commit 4021edc14c on Sep 5, 2017
UdjinM6 referenced this in commit 690cb58f80 on Sep 7, 2017
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels