[Github-comments] [geany/geany] Fix docbook parser memory leak (PR #3187)

[Jiří Techet]

I tested Geany with valgrind to make sure #3185 didn't introduce any memory leaks (it seems it didn't) and noticed this leak in the docbook parser (which we maintain by ourselves so no upstream patch needed). I'll merge this in a few days unless there are objections.

There are also leaks in Fortran (fixed upstream) and VHDL (different parser upstream) parsers which will get fixed once we merge the upstream versions.

For (my) future reference, I used valgrind with these parameters:
```
G_SLICE=always-malloc G_DEBUG=gc-friendly:resident-modules ./libtool --mode=execute valgrind --tool=memcheck --leak-check=full --leak-resolution=high --log-file=vgdump --suppressions=/usr/share/glib-2.0/valgrind/glib.supp src/geany
```
You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany/pull/3187

-- Commit Summary --

  * Fix docbook parser memory leak

-- File Changes --

    M ctags/parsers/geany_docbook.c (2)

-- Patch Links --

https://github.com/geany/geany/pull/3187.patch
https://github.com/geany/geany/pull/3187.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3187
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3187 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20220430/6e4a8e36/attachment.htm>