hash160 is defined in test_framework.script, so there is no need to define it here again.
qa: Remove duplicate `hash160` implementation #8521
pull laanwj wants to merge 1 commits into bitcoin:master from laanwj:2016_08_hash160_dupe changing 1 files +0 −3-
laanwj commented at 10:05 AM on August 16, 2016: member
-
c1582d5d85
qa: Remove duplicate `hash160` implementation
`hash160` is defined in `test_framework.script`, so there is no need to define it here again.
- laanwj added the label Tests on Aug 16, 2016
-
MarcoFalke commented at 10:22 AM on August 16, 2016: member
Agree that common code should be imported from the test_framework/ subdir. ACK c1582d5
-
jonasschnelli commented at 11:03 AM on August 16, 2016: contributor
-
paveljanik commented at 11:41 AM on August 16, 2016: contributor
utACK https://github.com/bitcoin/bitcoin/pull/8521/commits/c1582d5d8595ce8814a86dffdb0fa647e4100862
Three lines removed, good ;-)
-
sipa commented at 1:20 PM on August 16, 2016: member
Concept ACK
- laanwj merged this on Aug 17, 2016
- laanwj closed this on Aug 17, 2016
- laanwj referenced this in commit b41ac928b8 on Aug 17, 2016
- codablock referenced this in commit 80485226b9 on Sep 19, 2017
- codablock referenced this in commit a304b89f61 on Dec 29, 2017
- codablock referenced this in commit bdddabb3cf on Jan 8, 2018
- andvgal referenced this in commit 439b44e578 on Jan 6, 2019
- MarcoFalke locked this on Sep 8, 2021
Contributors
Labels