Subject: Re: Support for new links in megatools 1.10.2
From: Alberto Garcia
Date: Wed, 15 Apr 2020 12:25:25 +0200
On Wed, Apr 15, 2020 at 12:16:50PM +0200, Alberto Garcia wrote:
> > > Does the patch that I'm attaching look ok? (it took me a minute to
> > > write it, don't expect anything very spectacular :))
> > A small problem with that patch is that glib does allocate m1 even
> > on failed match. Anyway, looking at the glib code, that will just
> > produce a small memleak and nothing serious. I wouldn't worry about
> > it. Otherwise, it looks fine.
> Oh, I see. I suppose it's not so important since we're leaking all
> GRegex objects anyway?
Ah no, it uses auto cleanup, I wrote too quickly :-)
I may try to fix the leaks that my patch has then, I just wanted to
check if that was enough to support the new links.