<br><br><div class="gmail_quote">2011/11/14 Rafaël Carré <span dir="ltr"><<a href="mailto:funman@videolan.org">funman@videolan.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Le Sun, 13 Nov 2011 13:26:13 +0700,<br>
Sergey Radionov <<a href="mailto:rsatom@gmail.com">rsatom@gmail.com</a>> a écrit :<br>
<div class="im"><br>
> Patches needed for compilation of npapi and ActiveX under Visual Studio.<br>
<br>
</div>Patches 1, 2, and 4 applied, thanks.<br>
<br>
Patch 5 rejected, your gecko headers are out of date<br>
(this patch reverts d2e65e4462b9a634d5)<br>
<br>
$ svn co <a href="http://npapi-sdk.googlecode.com/svn/trunk/headers" target="_blank">http://npapi-sdk.googlecode.com/svn/trunk/headers</a><br>
$ svn log headers/npfunctions.h<br>
....<br>
<br>
r5 | <a href="mailto:josh.aas@gmail.com">josh.aas@gmail.com</a> | 2011-03-02 12:19:59 -0500 (mer. 02 mars 2011) | 1 ligne<br>
<br>
Make NP_GetMIMEDescription return value const. r=smorgan<br></blockquote><div>Yes, you are right. But then Jean-Baptiste Kempf need update <a href="http://people.videolan.org/%7Ejb/Contribs/contrib-20111017-win32-bin-gcc-4.4.4-runtime-3.17-only.tar.bz2">Contribs</a>, it contains outdated gecko heders.<br>
 <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
patch 3 and 6 (Visual Studio warnings) should be merged but I have a<br>
few remarks:<br>
<br>
There are a few float to long casts, shouldn't the interface be changed<br>
to take a float as input?<br>
fireOnMediaPlayerBufferingEvent, fireOnMediaPlayerTimeChangedEvent, etc<br></blockquote><div>And again, maybe you are right, I'll see closer.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<br>
About the "!=0" addition I think it would be clearer if you add a "!!"<br>
in front of the variable to make sure it is 0 or 1.<br>
Does that fix the warning?<br></blockquote><div>No, it is not work. I try to eliminate <br><i><br>warning C4800: 'int' : forcing value to bool 'true' or 'false' (performance warning)</i><br><br>But '!' (and therefore '!!') need bool argument, so it is not eliminate problem. <br>
So the more correct way (in C++) make bool from int, is compare it with zero.<br><br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
<br>
BTW could you have a look at the win64 patch I sent to the list? It was<br>
for you since you are the only one who knows activex I think...<br>
The patch is a bit ugly but if you have a better idea I am listening ^^<br></blockquote><div>Ok, I'll see it. And I'll make x64 support in my Visual Studio projects, to simplify testing.<br>But tell me please ID or Comment of you win64 patch.<br>
 <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><span class="HOEnZb"><font color="#888888">
<br>
--<br>
Rafaël Carré<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</font></span></blockquote></div><br>