[alsa-devel] strange code in pcm_route.c for 3 byte formats

Clemens Ladisch clemens at ladisch.de
Mon Nov 20 08:34:46 CET 2017


Maciej S. Szmigiero wrote:
> While reading alsa-lib sources I've spotted some strange code in
> src/pcm/pcm_route.c:
>        /* 3 bytes formats? */
>        route->params.use_getput =
>                (snd_pcm_format_physical_width(src_format) + 7) / 3 == 3 ||
>                (snd_pcm_format_physical_width(dst_format) + 7) / 3 == 3;

"(x + n-1) / n" is "divide by n while rounding up".

"/ 3" should have been "/ 8".


Regards,
Clemens


More information about the Alsa-devel mailing list