Hi Peter,

Your patch contains the following :
     PA_ECHO_CANCELLER_SPEEX = 0,
+#ifdef HAVE_SPEEX
+    PA_ECHO_CANCELLER_SPEEX,
+#endif
+#ifdef HAVE_ADRIAN_EC
     PA_ECHO_CANCELLER_ADRIAN,

This will lead to a bug if Speex is disabled and Adrian
or WRT is enabled, since ec_table will contain Adrian or WRT at index 0.

And also, you'll say I'm picky but when ./configure finishes,
it displays a summary with a lot of yes and no. The yes or
no corresponding to Adrian's is not vertically aligned with the
others.

Regards,
Frédéric
_______________________________________________
pulseaudio-discuss mailing list
pulseaudio-discuss@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss

Reply via email to