On Mon, Feb 02, 2015 at 11:49:29AM +0100, Peter Meerwald wrote:
> --- a/libavcodec/g722.c
> +++ b/libavcodec/g722.c
> @@ -71,6 +71,35 @@ const int16_t ff_g722_low_inv_quant6[64] = {
>
> +static inline void s_zero(int cur_diff, struct G722Band *band) {
{ on the next line for K&R style
> + int s_zero = 0;
> +
> + #define ACCUM(k, x, d) do { \
> + int tmp = x; \
> + band->zero_mem[k] = ((band->zero_mem[k]*255) >> 8) + \
> + d*((band->diff_mem[k]^cur_diff) < 0 ? -128 : 128); \
> + band->diff_mem[k] = tmp; \
> + s_zero += (tmp * band->zero_mem[k]) >> 15; \
> + } while (0)
The macro has an odd amount of indentation. I'd move it before the function
and add some spaces around operators.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
