cleanup include of assert.h #4781

pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:fix_assert changing 2 files +4 −2

Diapolo commented at 8:57 PM on August 28, 2014: none

No description provided.
jgarzik commented at 9:00 PM on August 28, 2014: contributor

ACK once the "fix" word is removed. This is a cleanup. It does not fix any behavior.

Cleanups should not be labelled as fixes.
cleanup include of assert.h 187115c01d
Diapolo renamed this:
~~fix include of assert.h~~
cleanup include of assert.h
on Aug 28, 2014
Diapolo commented at 9:03 PM on August 28, 2014: none

@jgarzik Agreed, thanks for mentioning. I changed it :).
jgarzik commented at 9:05 PM on August 28, 2014: contributor

ut ACK
BitcoinPullTester commented at 9:39 PM on August 28, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4781_187115c01d594bda3a4825ca45404ca27585b7f3/ for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
sipa commented at 12:39 AM on August 30, 2014: member

ut ACK
sipa merged this on Aug 30, 2014
sipa closed this on Aug 30, 2014
sipa referenced this in commit f5f1ab8dc3 on Aug 30, 2014
Diapolo deleted the branch on Aug 30, 2014
MarcoFalke locked this on Sep 8, 2021

Contributors