laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-tetra/+/29389 ) Change subject: only compute filter when opt_afc is enabled ...................................................................... only compute filter when opt_afc is enabled Change-Id: Icc42a70b07600267ccfad3969b738b3b4a606a61 --- M src/float_to_bits.c 1 file changed, 3 insertions(+), 6 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve diff --git a/src/float_to_bits.c b/src/float_to_bits.c index 89d0421..6948f24 100644 --- a/src/float_to_bits.c +++ b/src/float_to_bits.c @@ -81,13 +81,11 @@ int fd, fd_out, opt; int opt_verbose = 0; int opt_afc = 0; - float filter = 0; float filter_val = 0.0001; float filter_goal = 0; int sym; - while ((opt = getopt(argc, argv, "vaf:F:")) != -1) { switch (opt) { case 'v': @@ -141,11 +139,10 @@ rc /= sizeof(*fl); int i; for (i = 0; i < rc; ++i) { - - if ((fl[i] > -MAXVAL) && (fl[i] < MAXVAL)) { - filter = filter * (1.0 - filter_val) + (fl[i] - filter_goal) * filter_val; - } if (opt_afc) { + if ((fl[i] > -MAXVAL) && (fl[i] < MAXVAL)) { + filter = filter * (1.0 - filter_val) + (fl[i] - filter_goal) * filter_val; + } sym = process_sym_fl(fl[i]-filter); } else { sym = process_sym_fl(fl[i]); 3 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. -- To view, visit https://gerrit.osmocom.org/c/osmo-tetra/+/29389 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-tetra Gerrit-Branch: master Gerrit-Change-Id: Icc42a70b07600267ccfad3969b738b3b4a606a61 Gerrit-Change-Number: 29389 Gerrit-PatchSet: 6 Gerrit-Owner: wbokslag &lt;w.bokslag(a)midnightblue.nl&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria &lt;vyanitskiy(a)sysmocom.de&gt; Gerrit-Reviewer: laforge &lt;laforge(a)osmocom.org&gt; Gerrit-MessageType: merged