[Libreoffice] [PUSHED][patch] memset used on struc with std::vector / Windows code

Caolán McNamara caolanm at redhat.com
Tue Sep 13 05:37:21 PDT 2011


On Sun, 2011-09-11 at 15:48 +0200, Eike Rathke wrote:
> Hi Pierre-André,
> 
> On Sunday, 2011-09-11 14:20:57 +0200, Pierre-André Jacquod wrote:
> 
> > the struct DdeInstData contains a std::vector, but was initialized
> > with memset( pData,0,sizeof(DdeInstData) )
> 
> Good catch.

Yup, great catch. Thanks for this, pushed now.

> That should happen in a DdeInstData ctor instead.

I've moved it into the ctor now.

C.



More information about the LibreOffice mailing list