[PATCHv2 05/10] tests: Add remap test code to cpu-test

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> Thanks! I'll apply this soon (along with patches 1-4, the rest I haven't
> yet looked at). A couple of comments:

thank you for considering the patches!

> > + static void run_remap_test_mono_stereo_float(pa_remap_t *remap,
> > +    pa_do_remap_func_t func, pa_do_remap_func_t orig_func,
> > +    int align, pa_bool_t correct, pa_bool_t perf) {
> 
> Wrapped function argument lists are usually formatted like this:
> 
> static void run_remap_test_mono_stereo_float(
>         pa_remap_t *remap,
>         pa_do_remap_func_t func,
>         pa_do_remap_func_t orig_func,
>         int align,
>         pa_bool_t correct,
>         pa_bool_t perf) {
> 
> > +START_TEST (remap_sse_test) {
> 
> The test is for SSE2, so I'll change the test name to remap_sse2_test.

ack!

p.

-- 

Peter Meerwald
+43-664-2444418 (mobile)


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux