[x265] Some suggestions on x265 BitCost class

Steve Borho steve at borho.org
Thu Oct 8 22:12:39 CEST 2015


On 10/08, Ximing Cheng wrote:
> In the x265 BitCost class, s_bitsizes and s_costs[qp] are malloc by the
> new/delete operator, may change these into X265_MALLOC/X265_FREE?

agreed

> May s_bitsizes changes into s_bitsizes = new float[4 * BC_MAX_MV + 1] + 2 *
> BC_MAX_MV; in BitCost::CalculateLogs() and delete[] (s_bitsizes - 2 *
> BC_MAX_MV); in BitCost::destroy()  so that in inline uint32_t bitcost(const
> MV& mv) const and static inline uint32_t bitcost(const MV& mv, const MV&
> mvp), we can remove the abs operation.

I'm not following how the indexing could work, can you do this in a
patch?

-- 
Steve Borho


More information about the x265-devel mailing list