20 Nov
2017
20 Nov
'17
8:34 a.m.
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