[vlc-devel] [PATCH] 6 small npapi patch.
rsatom at gmail.com
Mon Nov 14 04:57:36 CET 2011
2011/11/14 Rafaël Carré <funman at videolan.org>
> Le Sun, 13 Nov 2011 13:26:13 +0700,
> Sergey Radionov <rsatom at gmail.com> a écrit :
> > Patches needed for compilation of npapi and ActiveX under Visual Studio.
> Patches 1, 2, and 4 applied, thanks.
> Patch 5 rejected, your gecko headers are out of date
> (this patch reverts d2e65e4462b9a634d5)
> $ svn co http://npapi-sdk.googlecode.com/svn/trunk/headers
> $ svn log headers/npfunctions.h
> r5 | josh.aas at gmail.com | 2011-03-02 12:19:59 -0500 (mer. 02 mars 2011) |
> 1 ligne
> Make NP_GetMIMEDescription return value const. r=smorgan
Yes, you are right. But then Jean-Baptiste Kempf need update
it contains outdated gecko heders.
> patch 3 and 6 (Visual Studio warnings) should be merged but I have a
> few remarks:
> There are a few float to long casts, shouldn't the interface be changed
> to take a float as input?
> fireOnMediaPlayerBufferingEvent, fireOnMediaPlayerTimeChangedEvent, etc
And again, maybe you are right, I'll see closer.
> About the "!=0" addition I think it would be clearer if you add a "!!"
> in front of the variable to make sure it is 0 or 1.
> Does that fix the warning?
No, it is not work. I try to eliminate
warning C4800: 'int' : forcing value to bool 'true' or 'false' (performance
But '!' (and therefore '!!') need bool argument, so it is not eliminate
So the more correct way (in C++) make bool from int, is compare it with
> BTW could you have a look at the win64 patch I sent to the list? It was
> for you since you are the only one who knows activex I think...
> The patch is a bit ugly but if you have a better idea I am listening ^^
Ok, I'll see it. And I'll make x64 support in my Visual Studio projects, to
But tell me please ID or Comment of you win64 patch.
> Rafaël Carré
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the vlc-devel