[PATCH] osmo-trx[master]: sigProcLib: Specify standard namespace for isnan()

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.org
Tue Jun 20 00:44:23 UTC 2017


Review at  https://gerrit.osmocom.org/2977

sigProcLib: Specify standard namespace for isnan()

In commit a3dce85f
"sigProcLib: Use explicit NaN check in sinc table generation"

Use of std::isnan(double) was added without namespace specifier,
which may cause build issues depending on whether the C version
isnan() call is available. Add standard namespace to force C++
call usage and potential build issues.

Change-Id: I49328c43fdd690a4e6a2b2e949411aaf5674ead1
---
M Transceiver52M/sigProcLib.cpp
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/77/2977/1

diff --git a/Transceiver52M/sigProcLib.cpp b/Transceiver52M/sigProcLib.cpp
index 9d1ef49..e94170b 100644
--- a/Transceiver52M/sigProcLib.cpp
+++ b/Transceiver52M/sigProcLib.cpp
@@ -982,7 +982,7 @@
   for (int i = 0; i < TABLESIZE; i++) {
     auto x = (double) i / TABLESIZE * 8 * M_PI;
     auto y = sin(x) / x;
-    sincTable[i] = isnan(y) ? 1.0 : y;
+    sincTable[i] = std::isnan(y) ? 1.0 : y;
   }
 }
 

-- 
To view, visit https://gerrit.osmocom.org/2977
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I49328c43fdd690a4e6a2b2e949411aaf5674ead1
Gerrit-PatchSet: 1
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Owner: Tom Tsou <tom at tsou.cc>



More information about the gerrit-log mailing list