This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
Tom Tsou gerrit-no-reply at lists.osmocom.orgTom Tsou has submitted this change and it was merged.
Change subject: Resampler: Fix initialization return checking
......................................................................
Resampler: Fix initialization return checking
Greater-than comparison was used on boolean type.
Change-Id: Ia3b71b3a06b34a6fd781bf197ecf9d5cc1711d13
---
M Transceiver52M/Resampler.cpp
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
  Tom Tsou: Looks good to me, approved
  Max: Looks good to me, but someone else must approve
  Jenkins Builder: Verified
diff --git a/Transceiver52M/Resampler.cpp b/Transceiver52M/Resampler.cpp
index 8a73b79..72b11a9 100644
--- a/Transceiver52M/Resampler.cpp
+++ b/Transceiver52M/Resampler.cpp
@@ -191,7 +191,7 @@
 bool Resampler::init(float bw)
 {
 	/* Filterbank filter internals */
-	if (initFilters(bw) < 0)
+	if (!initFilters(bw))
 		return false;
 
 	/* Precompute filterbank paths */
-- 
To view, visit https://gerrit.osmocom.org/2159
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia3b71b3a06b34a6fd781bf197ecf9d5cc1711d13
Gerrit-PatchSet: 2
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Owner: Tom Tsou <tom at tsou.cc>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Tom Tsou <tom at tsou.cc>