[MPlayer-users] Zoom problem using DirectX

[Reimar Döffinger]

On Thu, Sep 06, 2012 at 11:22:25AM -0700, Wayne wrote:
> I mention r33488 because that is the last version that I found to work in terms of being able to correctly adjust the aspect ratio (addressed and fixed previously) and adjust the zoom ratio of a playing video, i.e., being able to 'zoom in' on a portion of the screen.  I'm using the latest version of smplayer, with the keys "W" to zoom out (Zoom-) and "E" to zoom in (Zoom+).
> 
> Trying r33488 with smplayer should make it clear how it *should* work.  Unfortunately, DirectX is the only driver that works on some systems.  Anything after r33488 exhibits the 'zoom' problem mentioned, and other than exchanging the mplayer versions I made no other setting changes.  Running with Win32-sp1.

I suspect below patch to fix it, but I need to double-check it, plus find time (or someone)
to actually test on Windows.

Index: vo_directx.c
===================================================================
--- vo_directx.c	(revision 35143)
+++ vo_directx.c	(working copy)
@@ -470,8 +470,6 @@
         width   += vo_panscan_x;
         height  += vo_panscan_y;
     }
-    width    = FFMIN(width, vo_screenwidth);
-    height   = FFMIN(height, vo_screenheight);
     rd.left += (vo_dwidth - width) / 2;
     rd.top  += (vo_dheight - height) / 2;