Hi Jean-Pierre, Thank you for the notice. I checked the code, and I think your proposal is valid. I have already pushed a patch to fix the problem.
Regards, Orson On 09/07/2017 09:04 AM, jp charras wrote: > Hi Orson, > > Could you have a look into your commit > a3fc0ef658db69d601f7c91e41c91f43880c1ab0? > > It fixes a memory leak by adding and calling deleteTemplates() which delete > the footprint list in > memory. > > However, because it is called each time EAGLE_PLUGIN::init is called, the > loaded library is in fact > cleared as soon as you want to load a footprint (for instance in footprint > viewer) > > My opinion is deleteTemplates() should not be called inside init(), but I am > not very familiar with > the Eagle plugin code, so I let you to fix this issue. > > Thanks. > _______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : kicad-developers@lists.launchpad.net Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp
