[x265] [PATCH] Implementation for Intra-refresh

[Steve Borho]

On 09/15, Santhoshini Sekar wrote:
> On Mon, Sep 14, 2015 at 9:16 PM, Steve Borho <steve at borho.org> wrote:
> > On 09/14, santhoshini at multicorewareinc.com wrote:
> > >                  setSearchRange(cu, mvp, m_param->searchRange, mvmin,
> > mvmax);
> > > +                if (m_param->bIntraRefresh && m_slice->m_sliceType ==
> > P_SLICE)
> > > +                {
> > > +                    clipMvPIR(cu, mvmin);
> > > +                    clipMvPIR(cu, mvmax);
> > > +                }
> >
> > there is a lot of code duplication between setSearchRange() and
> > clipMvPIR(). Should the two be combined?
> >
> > is setSearchRange() called in more places?

> Yes setSearchRange() is called in many places wherein clipMvPIR() is
> not needed. Initially I had the functionality of clipMvPIR() in
> clipMV(). But, later added this new function as clipMV is also called
> in multiple places.

why is clipMvPIR not needed after these other setSearchRange() calls?

-- 
Steve Borho