[pulseaudio-discuss] [PATCH] volume-test: Increase the allowed number of rouding errors

Tanu Kaskinen tanu.kaskinen at linux.intel.com
Fri Jan 3 04:16:11 PST 2014


On Sat, 2013-12-14 at 09:21 +0200, Tanu Kaskinen wrote:
> ---
>  src/tests/volume-test.c | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/src/tests/volume-test.c b/src/tests/volume-test.c
> index a2daf3e..1ab0b5c 100644
> --- a/src/tests/volume-test.c
> +++ b/src/tests/volume-test.c
> @@ -138,7 +138,13 @@ START_TEST (volume_test) {
>      pa_log("max deviation: %lu n=%lu", (unsigned long) md, (unsigned long) mdn);
>  
>      fail_unless(md <= 1);
> -    fail_unless(mdn <= 251);
> +
> +    /* mdn counts the times there were rounding errors during the test. The
> +     * number of rounding errors seems to vary slightly depending on the
> +     * hardware. The original limit was 251 errors, but it was increased to 253
> +     * when the test was failing on Tanu's laptop.
> +     * See https://bugs.freedesktop.org/show_bug.cgi?id=72374 */
> +    fail_unless(mdn <= 253);
>  }
>  END_TEST
>  

No feedback received. I pushed this patch now.

-- 
Tanu



More information about the pulseaudio-discuss mailing list