[x265] [PATCH 1 of 2] aq : bug fix . extend border of TComPic::m_origPicYuv to a multiple of 16

[Steve Borho]

On Tue, Nov 5, 2013 at 12:18 PM, Steve Borho <steve at borho.org> wrote:

>
>
>
> On Tue, Nov 5, 2013 at 9:45 AM, <aarthi at multicorewareinc.com> wrote:
>
>> # HG changeset patch
>> # User Aarthi Thirumalai<aarthi at multicorewareinc.com>
>> # Date 1383666119 -19800
>> #      Tue Nov 05 21:11:59 2013 +0530
>> # Node ID 00a0690f4e3efb69f80a5750ea3a7f0ec493e013
>> # Parent  ece7af249573a3965d21589f8e1d54a8e1cd2ce9
>> aq : bug fix . extend border of TComPic::m_origPicYuv to a multiple of 16
>>      when aqMode is enabled.
>>
>
> I don't think we want to add the padding in this way; this makes the
> encoder actually encode out to the 16byte boundary, which makes extra work
> for the encoder.
>
> Instead, where the padding is done in copyFromPicture, I think it should
> just pad out past the pad value that was calculated here.
>

*and* the padding code in lowres::init() needs to be removed, and the logic
which calculates the number of 16x16 blocks (or lowres 8x8 blocks) needs to
be made consistent.

-- 
Steve Borho
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20131105/d3eded39/attachment.html>