-
Notifications
You must be signed in to change notification settings - Fork 286
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
refactor: Add mem
module to allow tests to override allocators.
#2393
Conversation
d194b00
to
f68de74
Compare
mem
module to allow tests to override allocators. [WIP]mem
module to allow tests to override allocators.
ec51cce
to
8062297
Compare
eb51056
to
fe50f6c
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #2393 +/- ##
===========================================
+ Coverage 9.99% 74.49% +64.49%
===========================================
Files 141 87 -54
Lines 32530 26194 -6336
===========================================
+ Hits 3250 19512 +16262
+ Misses 29280 6682 -22598
☔ View full report in Codecov by Sentry. |
554e7cd
to
bfb95ae
Compare
51170fc
to
5e7a651
Compare
f4a6b4e
to
6416aa7
Compare
9c26abb
to
ea5e1e3
Compare
944cc83
to
63dc3e4
Compare
This will allow us to do more interesting things with memory allocation within toxcore, and allow fuzzers to explore various allocation failure paths.
A bit confused by this PR and #2397 both introducing the mem module and using new system_{random,network,memory}. Is one PR build on top of another? Or perhaps these are alternative implementations of the same thing? |
This will allow us to do more interesting things with memory allocation within toxcore, and allow fuzzers to explore various allocation failure paths.
This change is