That is good news, thanks I will post the call stack when I find the crash again.
On Wed, Dec 30, 2009 at 6:29 PM, Armin Le Grand <[email protected]> wrote: > Huaidong Qiu schrieb: >> >> Sorry, the second one should be >> >> 2) The virtual deconstructor does not get called properly. > > Sorry to tell You, but this code is used since DEV300m30, so i would have > stumbled above this when this should be the case. > > Maybe You should also take into account that BasePrimitive2D is a UNO API > implementation class derived from cppu::WeakComponentImplHelper1 which > provides a virtual destructor in the template base class anyways. > > When You want to do more with Primitives, please refer to (wait for > integration or get) CWS aw079 where more documentation is added to > drawinglayer header files. It is essential to know about the priciples used > for handling primitives. > > HTH! > >> >> On Wed, Dec 30, 2009 at 2:37 AM, Huaidong Qiu <[email protected]> >> wrote: >>> >>> I encounter some crash on impr sometime, which looks like some >>> BasePrimitive2D derived objects do not properly released, but I am not >>> sure about this. >>> >>> I think maybe below elements should be taken into consideration. >>> >>> 1) STL container's memory manager does not call the deconstructer , it >>> just cashes the allocated memory. >>> 2) The virtual deconstructor does called properly. >>> >>> On Tue, Dec 29, 2009 at 11:22 PM, Armin Le Grand <[email protected]> >>> wrote: >>>> >>>> Huaidong Qiu schrieb: >>>>> >>>>> Hi all, >>>>> >>>>> Why BasePrimitive2D does not have a virtual deconstructor? >>>> >>>> It will have one after aw078 integration where i optimized some >>>> primitive >>>> stuff and added a whole bunch of documentation. >>>> >>>> Will the compiler >>>>> >>>>> give it one? >>>> >>>> Yes. >>>> >>>>> So many classes derived from it. >>>> >>>> Is this good or bad from Your POV...? >>>> >>>>> Thanks in advance >>>>> >>>>> -Huai Dong >>>>> >>>> --------------------------------------------------------------------- >>>> To unsubscribe, e-mail: [email protected] >>>> For additional commands, e-mail: [email protected] >>>> >>>> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> For additional commands, e-mail: [email protected] >> >> >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
