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".

// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to