On Sun, Aug 08, 2004 at 09:14:29PM +0200, Stefan Lucke wrote:
> On Sonntag, 8. August 2004 19:24, Ville Syrj�l� wrote:
> > On Sun, Aug 08, 2004 at 05:01:48PM +0200, Stefan Lucke wrote:
> 
> [ .. ]
> 
> > > Vertical scaled crop is still not right. Displayed area is shifted a bit up
> > > (top black bar is smaller than bottom one) and displayed content is shifted up 
> > > too.
> > 
> > Does the attached patch fix everything (or even something :) ?
> 
> Passed all of my test cases, so it fixes _everything_ :-) . 

Great. I commited it with the following additions.

+     field_height = surface->height;
+
      if (config->options & DLOP_DEINTERLACING) {
+          field_height /= 2;
...
-     mbes->regs.besV1SRCLST = mbes->regs.besV2SRCLST = source.h - 1;
+     mbes->regs.besV1SRCLST = mbes->regs.besV2SRCLST =
+          field_height - 1 - source.y - (croptop >> 16);

This shouldn't cause any visible changes but it matches what the specs 
say.

-- 
Ville Syrj�l�
[EMAIL PROTECTED]
http://www.sci.fi/~syrjala/


Reply via email to