-
Notifications
You must be signed in to change notification settings - Fork 37.7k
test: Remove unused txmempool include from tests #26286
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The head ref may contain hidden characters: "2210-test-cleanup-\u{1F43C}"
Conversation
Follow-up to #26250 |
Concept ACK - looks like some previously transitively included includes will need to be added: test/util_tests.cpp: In member function ‘void util_tests::util_seed_insecure_rand::test_method()’:
test/util_tests.cpp:1441:33: error: ‘sqrt’ was not declared in this scope
1441 | int err = 30*10000./mod*sqrt((1./mod*(1-1./mod))/10000.);
| ^~~~
make[2]: *** [Makefile:18499: test/test_bitcoin-util_tests.o] Error 1 |
Concept ACK. |
fa407d1
to
fae702d
Compare
I can reduce the fuzz compile time (user+kernel time) by 20% with a dirty
|
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ConflictsReviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
Concept ACK |
Does anyone know why msvc is unable to link this? |
@sipsorcery @hebasto any thoughts? |
--- a/build_msvc/test_bitcoin-qt/test_bitcoin-qt.vcxproj
+++ b/build_msvc/test_bitcoin-qt/test_bitcoin-qt.vcxproj
@@ -54,6 +54,9 @@
<ProjectReference Include="..\libbitcoin_zmq\libbitcoin_zmq.vcxproj">
<Project>{792d487f-f14c-49fc-a9de-3fc150f31c3f}</Project>
</ProjectReference>
+ <ProjectReference Include="..\libtest_util\libtest_util.vcxproj">
+ <Project>{1e065f03-3566-47d0-8fa9-daa72b084e7d}</Project>
+ </ProjectReference>
<ProjectReference Include="..\libleveldb\libleveldb.vcxproj">
<Project>{18430fef-6b61-4c53-b396-718e02850f1b}</Project>
</ProjectReference> |
872982e
to
fad7f22
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK fad7f22
Co-authored-by: Aurèle Oulès <aurele@oules.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reACK 1c48dae
Thanks for the co-author.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 1c48dae, I have reviewed the code and it looks OK, I agree it can be merged.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 1c48dae
Seems odd to include this heavy header in all tests despite it only being used in a few tests.
Can be reviewed with
--color-moved=dimmed-zebra --ignore-all-space