[x265] [PATCH] stats: fix use of uninitialized pixels in maximum and average luma level calculation

Steve Borho steve at borho.org
Fri Aug 14 06:31:46 CEST 2015


On 08/14, Deepthi Nandakumar wrote:
> I did a double take on the if (padx) part as well, then figured it works
> since padx is guaranteed to be non-zero (because of padx++).
> 
> Min, can you do a quick profile on whether it is better to do this outside
> the padx loop, for the whole frame, or inside the padx loop, one row at a
> time?

FWIW, the function would work just fine if the "if (padx)" line were
simply removed. The for() loop over padx would be a nop if it was zero.

-- 
Steve Borho


More information about the x265-devel mailing list