Hi, On Fri, Jul 8, 2016 at 3:00 PM, Martin Storsjö <[email protected]> wrote:
> On Fri, 8 Jul 2016, Ronald S. Bultje wrote: > > Hi, >> >> On Thu, Jul 7, 2016 at 7:08 PM, Janne Grunau <[email protected]> >> wrote: >> >> +#define SRC_BUF_STRIDE 32 >>>> +#define SRC_BUF_SIZE ((size + 5) * SRC_BUF_STRIDE) >>>> +// The 2 * stride + 2 offset is necessary to avoid reading out of >>>> >>> bounds, >>> >>> that sounds a little misleading. just stating that the mc sub pixel >>> interpolation filter needs 2 previous pixels in either direction would >>> be clearer. Feel free to ignore >>> >> >> >> It needs 2 before and 3 after, not 2+2. >> > > What I mostly refer to here is that it needs 2 before both horizontally > and vertically, explaining the "+2 * SRC_BUF_STRIDE + 2". Right, I misread Janne's comment, sorry about that. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
