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

Valgrind CI pass #14

Open
frozencemetery opened this issue Sep 2, 2020 · 0 comments
Open

Valgrind CI pass #14

frozencemetery opened this issue Sep 2, 2020 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@frozencemetery
Copy link
Member

The test suite supports running under valgrind if additional flags are passed. Two tests are known to not work (Simo believes this is because they rely on program name matching). So we would need a separate pass.

We would also want to generate a suppressions file for libverto/libev (but this is probably not worth it - these are fixed-size one-time leaks, and valgrind has trouble generating useful traces for it).

@frozencemetery frozencemetery added enhancement New feature or request good first issue Good for newcomers labels Sep 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant