[MPlayer-dev-eng] [PATCH] Border processing adaptive quant, Take 2

[Guillaume POIRIER]

Hi,

On 3/22/05, Guillaume Poirier <poirierg at gmail.com> wrote:
> Hi,
> 
> 
> On Tue, 22 Mar 2005 15:10:04 -0400, Giacomo Comes <comes at naic.edu> wrote:
> > On Tue, Mar 22, 2005 at 07:50:33PM +0100, Guillaume POIRIER wrote:
> > > Hi,
> > > This patch is supposed to add support to border processing adaptive
> > > quant to lavc in MEncoder.
> > > This functionality was added to ffmpeg in late January by Christophe
> > > Massiot, but so far, there were no MEncoder support, hence this patch.
> > >
> > > The problem is that this patch doesn't work, and even though it's very
> > > simple, I don't quite understand what's the problem.
> > >
> > > Any bright mind to light me up?
> >
> > May be because you give as valid range 0.01, 255.0 but you assign
> > 0.0 as initial value?
> 
> That's a thought. I have to admit I set this range without knowing
> which valid range it supported.

Ok, here's a new try, which works. I feel so confident that it doesn't
break anything and that if it does, I'll be able to fix it, that I'm
gonna commit it soon.

The description I'll add to the man page will look like that:

border_masking=<0.0\-1.0>
border-processing for MPEG-style encoders.
Border processing increases the quantizer for macroblocks which are less
than 1/5th far from the border of the frame, since they are visually
less important.

What do you think?

Guillaume
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ve_lavc_BorderMasking.patch
Type: text/x-patch
Size: 1408 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20050508/e81bc578/attachment.bin>