På Mandag 28 juli 2008 , 12:14:44 skrev Alexandre Julliard:
> Alexander Nicolaysen Sørnes <[EMAIL PROTECTED]> writes:
> > +
> >  static const IDirectPlay8PeerVtbl DirectPlay8Peer_Vtbl =
> >  {
> >      IDirectPlay8PeerImpl_QueryInterface,
> >      IDirectPlay8PeerImpl_AddRef,
> >      IDirectPlay8PeerImpl_Release,
> >      IDirectPlay8PeerImpl_Initialize,
> > -    IDirectPlay8PeerImpl_EnumServiceProviders
> > +    IDirectPlay8PeerImpl_EnumServiceProviders,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    NULL,
> > +    IDirectPlay8PeerImpl_Close
>
> Don't do that, partial vtables lead to hard to find crashes. The vtable
> should be completely initialized from the start with stub functions.

Thanks, I will send a new patch.


Alexander N. Sørnes


Reply via email to