[vlc-devel] [PATCH] npapi: zero NPClass before using.

Sergey Radionov rsatom at gmail.com
Thu Dec 1 10:10:26 CET 2011


2011/12/1 Jean-Baptiste Kempf <jb at videolan.org>

> On Thu, Dec 01, 2011 at 04:06:35PM +0700, Sergey Radionov wrote :
> > 2011/12/1 Jean-Baptiste Kempf <jb at videolan.org>
> >
> > > On Thu, Dec 01, 2011 at 09:13:06AM +0700, Sergey Radionov wrote :
> > > >
> > >
> > > Wouldn't setting the unset parameters to NULL be faster, and more
> > > elegant?
> > >
> > Maybe, but I am take care about possible future adding of new members to
> > this struct.
>
> The probability of extension of NPClass is quite low...
>
> > But I am not insist. So what you say, should I replace it with simple
> > assignment?
>
> Well, seeing the doc and the headers, almost all members are init'd, but
> a few ones. Setting them as NULL might be enough (and maybe not, so this
> approach would be needed).
>
Ok, as you wish, I'll make new patch, not problem.


>
> Best regards,
>
> --
> Jean-Baptiste Kempf
> http://www.jbkempf.com/ - +33 672 704 734
> Sent from my Electronic Device
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20111201/7cfe641a/attachment.html>


More information about the vlc-devel mailing list