On 17/07/15 21:21, Henrik Gramner wrote: > 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 >
Thanks for spotting, you are right, amending now =) lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
