Get rid of a compiler warning due to #if 0'd test #8003

pull avar wants to merge 1 commits into bitcoin:master from avar:fix-unused-function-compiler-warning changing 1 files +1 −1

avar commented at 10:11 PM on May 4, 2016: none
In v0.9.0rc2-4332-g8f0d79e the src/test/scheduler_tests.cpp test was disabled, but it was done in such a way as to define a function that then was no longer used, leading to this warning under at least my GCC 4.9.2-10:
```
test/scheduler_tests.cpp:32:13: warning: ‘void
scheduler_tests::MicroSleep(uint64_t)’ defined but not used
[-Wunused-function]
```
Fix this by just moving the "#if 0" a few lines up so that it also defines out a function used only by this test.

Get rid of a compiler warning due to #if 0'd test

In v0.9.0rc2-4332-g8f0d79e the src/test/scheduler_tests.cpp test was
disabled, but it was done in such a way as to define a function that
then was no longer used, leading to this warning under at least my GCC
4.9.2-10:

    test/scheduler_tests.cpp:32:13: warning: ‘void
    scheduler_tests::MicroSleep(uint64_t)’ defined but not used
    [-Wunused-function]

Fix this by just moving the "#if 0" a few lines up so that it also
defines out a function used only by this test.

abf01275a7

avar commented at 10:35 PM on May 4, 2016: none

B.t.w. the failed travis run seems to have to do with some generic travis issue instead of an issue with this patch, the tests stopped after getting no output for 10 minutes.
paveljanik commented at 5:23 AM on May 5, 2016: contributor

See #7169.
laanwj commented at 5:51 AM on May 5, 2016: member

This is the easy way out. Please fix the test instead. (see See #7144, #6540)
laanwj added the label Tests on May 5, 2016
avar referenced this in commit b5d33fd6f1 on May 5, 2016
avar closed this on May 5, 2016
DrahtBot locked this on Sep 8, 2021

Contributors

Labels

Linked (view graph)

#8005 Add a comment indicating that the btc devs don't want a warning fixed