[PATCH wayland 6/6] tests: Remove memory leak checking infrastructure

Daniel Stone daniel at fooishbar.org
Thu Aug 30 12:54:50 UTC 2018


Hi,

On Wed, 29 Aug 2018 at 17:37, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> On 29 August 2018 at 07:17, Daniel Stone <daniels at collabora.com> wrote:
> > There are far better ways to detect memory leaks, such as either
> > valgrind or ASan. Having Meson makes it really easy to use these tools
> > in our tests, and we can do that in CI as well.
> >
> > Having these local wrappers actually completely broke ASan usage, so
> > remove them in favour of using the more powerful options.
>
> Fwiw adding valgrind to any build-system is a 2-5 line task ;-)

Sure, I'll happily review such a patch.

> Sanitizers on the other hand (esp. on shared libraries) is a pain.

Yes, I spent at least a couple of hours banging my head against that
particular brick wall last weekend, and decided not to.

Cheers,
Daniel


More information about the wayland-devel mailing list