Re: [RESOLVED][PATCH] Remove unused code (ImpSvtData)

2012-04-25 Thread Caolán McNamara
On Tue, 2012-04-24 at 15:02 -0700, Javier Catala wrote: > Hi everyone, > > this patch removes the unused destructor in the ImpSvtData class, as listed > in unusedcode.easy See http://lists.freedesktop.org/archives/libreoffice/2012-April/030785.html for the gotcha with unused destructors. I'll re

Re: [RESOLVED][PATCH] Remove unused code (ImpSvtData)

2012-04-25 Thread Caolán McNamara
On Wed, 2012-04-25 at 13:47 +0100, Caolán McNamara wrote: > We should try and fix the leak and call the destructor somewhere rather > than removing the destructor. Refactored now to not leave that gotcha in the unusedcode.easy list. Fixed. C. ___ Libre