Pádraig Brady wrote: > valgrind was complaining about definite mem leaks in `join` > so I refactored the offending code a little and removed the > inconseqential leak (which may mask real leaks in future). ... > Subject: [PATCH] maint: fix an inconsequential mem leak in join > > * src/join.c (join): Refactor the code that checks for misorder > at the tail of the files. The most significant change here is > that freeline() is called thus silencing a valgrind warning about > a definite but inconseqential memory leak. > (freeline): Make more general by doing nothing when passed NULL, > and setting freed pointers to NULL.
Thanks. That had been an annoyance when looking at leak summaries.
