Skip to content
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

Sanitizers #102

Draft
wants to merge 14 commits into
base: main
Choose a base branch
from
Draft

Sanitizers #102

wants to merge 14 commits into from

Conversation

Alex-PLACET
Copy link
Collaborator

No description provided.

@Alex-PLACET Alex-PLACET force-pushed the sanitizer-ci branch 4 times, most recently from eb7c5a8 to 9fcc237 Compare June 18, 2024 09:49
@Alex-PLACET Alex-PLACET force-pushed the sanitizer-ci branch 4 times, most recently from c4693b7 to 34b78be Compare June 26, 2024 12:06
@Alex-PLACET Alex-PLACET marked this pull request as ready for review June 26, 2024 12:11
@Alex-PLACET Alex-PLACET force-pushed the sanitizer-ci branch 2 times, most recently from 4890784 to cc706bc Compare July 10, 2024 07:39
.github/workflows/linux.yml Outdated Show resolved Hide resolved
.github/workflows/linux.yml Outdated Show resolved Hide resolved
.github/workflows/linux.yml Outdated Show resolved Hide resolved
include/sparrow/allocator.hpp Outdated Show resolved Hide resolved
include/sparrow/allocator.hpp Outdated Show resolved Hide resolved
@Alex-PLACET Alex-PLACET self-assigned this Jul 11, 2024
@Klaim
Copy link
Collaborator

Klaim commented Jul 16, 2024

Could you add a commit adding a memory issue to check that the report works as intended? We'll drop the commit after we see the report.

@JohanMabille
Copy link
Collaborator

Could you add a commit adding a memory issue to check that the report works as intended? We'll drop the commit after we see the report.

Was it done (and the commit has already been removed) ? If so, I think we can merge this PR.

@Alex-PLACET
Copy link
Collaborator Author

Alex-PLACET commented Sep 2, 2024

It seems that we have an issue with the logs file. If you check the tests execution, you will see several leaks. But the log file is not created. Let me check that first.

@Alex-PLACET Alex-PLACET marked this pull request as draft December 17, 2024 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants