[vlc-devel] [PATCH] ActiveX/Mozilla FullScreen support

[Jean-Baptiste Kempf]

Hello Sergey,

On Sun, May 22, 2011 at 09:10:00PM +0700, Sergey Radionov wrote :
> based on git://git.videolan.org/vlc/vlc-1.1.git

This is really welcomed!

> P.S. This is my first patch, so, may be some mistakes...

Hmm, first, tabs and trailing spaces are forbidden in the code.

Then you comment the warning about NPN_PluginThreadAsyncCall, why?

Finally, what are the bmp for?

Best Regards,

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device