On Fri, Jul 17, 2015 at 8:08 PM, Luca Barbato <[email protected]> wrote:
> -        qpel_mc_func (*tab)[16] = op ? h.avg_h264_qpel_pixels_tab : 
> h.put_h264_qpel_pixels_tab;
> +        qpel_mc_func(*tab)[16] = op ? h.avg_h264_qpel_pixels_tab : 
> h.put_h264_qpel_pixels_tab;

No space between type and identifier? I don't particularly have any
preference for either way, but it's used in most other places:
https://git.libav.org/?p=libav.git&a=search&h=HEAD&st=grep&s=[_a-zA-Z0-9]%2B+%2B\%28\*[^%2C]%2B\%29\[.*\]&sr=1

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

Reply via email to