pmaillot / X32-Behringer

This GIT repo (C language) holds applications and utilities for the Behringer X32 and M32 mixing consoles. Additional details, documentation, implementation examples and apps can be found in my website:
https://sites.google.com/site/patrickmaillot/x32
234 stars 41 forks source link

logic error in fxparse.c #4

Closed kmitch95120 closed 8 years ago

kmitch95120 commented 8 years ago

On line 203 of fxparse.c there's a test for PQ5 when the test should be for PQ5S since the "S" is the dual effect with two sets of parameters.

Here's the diff: diff --git a/fxparse.c b/fxparse.c index c268a1b..b1bab91 100644 --- a/fxparse.c +++ b/fxparse.c @@ -200,7 +200,7 @@ int XPQ5_PQ5S(char* buf, int k, int ifx, int PQ5S, int PQ5) { list2int(l_qhlfreq, ll_qhlfreq); // hi freq [1k5, 2k, 3k, 4k, 5k] ratio2float(10.); // hi boost [0, 10] toggle2int("OFF"); // transformer [OFF / ON]

pmaillot commented 8 years ago

Thanks a lot!,

Fixed J

-Patrick

From: kmitch95120 [mailto:notifications@github.com] Sent: Wednesday, July 13, 2016 11:51 PM To: pmaillot/X32-Behringer Subject: [pmaillot/X32-Behringer] logic error in fxparse.c (#4)

On line 203 of fxparse.c there's a test for PQ5 when the test should be for PQ5S since the "S" is the dual effect with two sets of parameters.

Here's the diff: diff --git a/fxparse.c b/fxparse.c index c268a1b..b1bab91 100644 --- a/fxparse.c +++ b/fxparse.c @@ -200,7 +200,7 @@ int XPQ5_PQ5S(char* buf, int k, int ifx, int PQ5S, int PQ5) { list2int(l_qhlfreq, ll_qhlfreq); // hi freq [1k5, 2k, 3k, 4k, 5k] ratio2float(10.); // hi boost [0, 10] toggle2int("OFF"); // transformer [OFF / ON]

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/pmaillot/X32-Behringer/issues/4 , or mute the thread https://github.com/notifications/unsubscribe/APOZ_mRy-ABTljn3K1JRh7FemWAnoxb6ks5qVV3OgaJpZM4JL4qH . https://github.com/notifications/beacon/APOZ_ns5zEn0EX-sdL-UTmi2-hyOtPbTks5qVV3OgaJpZM4JL4qH.gif

kmitch95120 commented 8 years ago

Works as expected now.