Remove redundant semicolons in Python code #9607

pull practicalswift wants to merge 1 commits into bitcoin:master from practicalswift:remove-redundant-semicolons-in-python-code changing 8 files +16 −16

practicalswift commented at 9:56 PM on January 20, 2017: contributor

No description provided.
Remove redundant semicolons in Python code 5cdf10611f
fanquake commented at 11:45 PM on January 20, 2017: member

trivial ACK 5cdf106 btw, this is also something that could have been combined into #9581.

in contrib/devtools/clang-format-diff.py:None in 5cdf10611f

 127 | @@ -128,7 +128,7 @@ def main():
 128 |          line_count = int(match.group(3))
 129 |        if line_count == 0:
 130 |          continue
 131 | -      end_line = start_line + line_count - 1;
 132 | +      end_line = start_line + line_count - 1

MarcoFalke commented at 11:47 PM on January 20, 2017:

Just a note that this exists upstream. Not against removing it here, though.

MarcoFalke commented at 11:47 PM on January 20, 2017: member

utACK 5cdf10611fe5ea77d4632c24e69e68aa9b8119e0
paveljanik commented at 7:58 AM on January 21, 2017: contributor

utACK https://github.com/bitcoin/bitcoin/commit/5cdf10611fe5ea77d4632c24e69e68aa9b8119e0
laanwj commented at 6:28 AM on January 23, 2017: member

Please, do not open a ton of pull requests to do minor style stuff changes for the test scripts. If those are necessary at all I'd prefer to see them combined over time (there's no hurry for these).
practicalswift commented at 8:28 AM on January 23, 2017: contributor

@laanwj OK, understood! Thanks for clarifying :-)
jonasschnelli approved
jonasschnelli commented at 9:03 AM on January 23, 2017: contributor

utACK 5cdf10611fe5ea77d4632c24e69e68aa9b8119e0
laanwj merged this on Jan 23, 2017
laanwj closed this on Jan 23, 2017
laanwj referenced this in commit 5cf3c60fcc on Jan 23, 2017
ryanofsky cross-referenced this on Feb 2, 2017 from issue Use importmulti timestamp when importing watch only keys (on top of #9682) by ryanofsky
jnewbery cross-referenced this on Apr 1, 2017 from issue Remove unused import. Remove accidental trailing semicolons. by practicalswift
codablock referenced this in commit 593b6dba9c on Jan 19, 2018
codablock referenced this in commit b078ee050a on Jan 20, 2018
codablock referenced this in commit 96b4910fbe on Jan 21, 2018
andvgal referenced this in commit 3573cdf629 on Jan 6, 2019
CryptoCentric referenced this in commit 937a53f63a on Feb 27, 2019
practicalswift deleted the branch on Apr 10, 2021
bitcoin locked this on Aug 16, 2022

Contributors

Linked (view graph)

#9108 Use importmulti timestamp when importing watch only keys (on top of #9682)#9581 [pep-8] Prefer "foo is None" to "foo == None". Prefer "foo not in bar" to "not foo in bar".#10137 Remove unused import. Remove accidental trailing semicolons.