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

Sergey Radionov rsatom at gmail.com
Mon May 23 11:04:23 CEST 2011



23.05.2011 13:57, 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.
sorry, accepted.

>
> Then you comment the warning about NPN_PluginThreadAsyncCall, why?
>
> Finally, what are the bmp for?
bmp files used on buttons in fullscreen control panel (yes, full screen control panel added too ;).
I place this button images directly to projects folder, because it using in both activeX and Mozilla 
modules. If it wrong, hint me how make in right. May be, make some sort of "shared" folder?

>
> Best Regards,
>

-- 
With best wishes,
Sergey Radionov



More information about the vlc-devel mailing list