Re: [PATCH][Trac 2311] codec_path mem leak
Reimar Döffinger <[email protected]>
| Newsgroups | gmane.comp.video.mplayer.devel |
|---|---|
| Message-ID | <[email protected]> |
On Fri, Jan 13, 2017 at 10:14:43PM +0100, Alexander Strasser wrote: > Hello all, > > while investigating the leaks pointed out in ticket #2311, I found > that the attached patch fixes the remaining leak. > > The current code seems pretty wrong. At least, as I understand it, > the function set_codec_path ever only got called once. The implementation > combined with exporting the codec_path global var seems not solid at all. > > Still I feel like I am missing something... Same feeling here, but I'd say go for it. _______________________________________________ MPlayer-dev-eng mailing list [email protected] https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng