<div>Sorry, I don't quite understand. You mean wingdi module should be integrated into directx video_output module? But as far as I understand, wingdi module is also intended for WinCE and therefore some of it's logic may be quite different from other directx code logic. To be honest I have spent some time getting acquainted with these modules, but I'm still not that familiar with them, so you probably better discuss that with other vlc developers, who have created those different modules.
</div>
<div>Or do you mean that my patch should unify code for switching fullscreen mode between those two modules? Theoretically it could be done, but they currently do not have common files, so I don't know how they can share this code.
<br> </div>
<div><span class="gmail_quote">On 5/8/07, <b class="gmail_sendername">Damien Fouilleul</b> <<a href="mailto:damien.fouilleul@laposte.net">damien.fouilleul@laposte.net</a>> wrote:</span>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">i'm not so warm about the wingdi patch, i'd rather have it merged with<br>the rest of the other modules so that integration is fully complete
<br><br>Sergey Volk wrote:<br>> Great! Thanks for reviewing and applying this patch.<br>> You may also want to have a look at a similar patch for wingdi module,<br>> which I have posted to vlc-devel mail-list on May 3.
<br>><br>><br>> On 5/8/07, *Damien Fouilleul* <<a href="mailto:damien.fouilleul@laposte.net">damien.fouilleul@laposte.net</a><br>> <mailto:<a href="mailto:damien.fouilleul@laposte.net">damien.fouilleul@laposte.net
</a>>> wrote:<br>><br>>     Sergey Volk wrote:<br>>     ><br>>     > Hi,<br>>     > Current implementation of directx video output modules (directx,<br>>     > direct3d, glwin32) leaves main interface window visible when the
<br>>     user<br>>     > toggles fullscreen mode. This causes two problems: (a) the user sees<br>>     > empty main interface window when he switches to another<br>>     application or<br>>     > cycles through existing windows using Alt-Tab and (b) main interface
<br>>     > window overlaps video window when he toggles "video-on-top" option<br>>     > while fullscreen mode is active. In order to fix both these<br>>     issues, we<br>>     > should hide main interface window when entering fullscreen mode and
<br>>     > restore it when leaving fullscreen mode. But some GUI modules (in<br>>     > particular qt4 module) may need to call show() for main interface<br>>     > window in order to apply "video-on-top" flag, so in order to avoid
<br>>     > condition (b) described earlier we need also to hide<br>>     > VOUT_SET_STAY_ON_TOP queries from main interface window while we<br>>     are<br>>     > in fullscreen mode and update "video-on-top" status for main window
<br>>     > when we are leaving fullscreen mode.<br>>     ><br>>     > I have also noticed that there is a code in<br>>     > src/video_output/vout_intf.c:FullscreenCallback (line 1151), which<br>>     > sets video-on-top variable to false, when entering fullscreen
<br>>     mode. I<br>>     > think this code should be removed, there is no point in disabling<br>>     > video-on-top option, as the user may change this option in<br>>     fullscreen<br>>     > mode anyway (through mouse / hotkey controls, at least). So
<br>>     > video_output modules should be aware that video-on-top variable<br>>     may be<br>>     > toggled in fullscreen mode and should handle such situations<br>>     > appropriately. I'm developing under win32/cygwin, so I haven't
<br>>     checked<br>>     > if similar reasoning applies to X11 module or any of the other video<br>>     > output modules. It seems that at least wingdi output module has the<br>>     > same problem, but it's code is somewhat different from
<br>>     > DirectX/Direct3D/Win32 OpenGL code, so I'll provide patch for wingdi<br>>     > module later. In the meantime, I think it is safe to leave that old<br>>     > code as it is.<br>>     >
<br>>     > Two patches are attached to this message:<br>>     vlc-vout-dx-fullscreen.patch<br>>     > is a version which fixes fullscreen mode for DirectX module only, it<br>>     > is intended only to demonstrate what changes are being applied to
<br>>     > current version. Another one - vlc-vout-dx-fullscreen2.patch , does<br>>     > essentially the same, only it fixes all three modules (directx,<br>>     > direct3d and opengl) and refactors some common code for toggling
<br>>     > fullscreen mode into Win32ToggleFullscreen function, to remove code<br>>     > duplication between those modules.<br>>     ><br>>     patch applied thanks !<br>><br>>     --<br>>     This is the vlc-devel mailing-list, see 
<a href="http://www.videolan.org/vlc/">http://www.videolan.org/vlc/</a><br>>     To unsubscribe, please read <a href="http://developers.videolan.org/lists.html">http://developers.videolan.org/lists.html</a><br>><br>
><br><br>--<br>This is the vlc-devel mailing-list, see <a href="http://www.videolan.org/vlc/">http://www.videolan.org/vlc/</a><br>To unsubscribe, please read <a href="http://developers.videolan.org/lists.html">http://developers.videolan.org/lists.html
</a><br><br></blockquote></div><br>