Change in libosmocore[master]: Drop old BSC references in fd check configure option

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/.

laforge gerrit-no-reply at lists.osmocom.org
Mon May 11 17:24:16 UTC 2020


laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/18169 )

Change subject: Drop old BSC references in fd check configure option
......................................................................

Drop old BSC references in fd check configure option

Change-Id: I053c2bfe461aa82085e7dac1cdcc95dd77219949
---
M configure.ac
M src/select.c
2 files changed, 3 insertions(+), 3 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  fixeria: Looks good to me, but someone else must approve
  osmith: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/configure.ac b/configure.ac
index cba0a41..ac887a0 100644
--- a/configure.ac
+++ b/configure.ac
@@ -227,12 +227,12 @@
 AC_ARG_ENABLE(bsc_fd_check,
 	[AS_HELP_STRING(
 		[--enable-bsc-fd-check],
-		[Instrument bsc_register_fd to check that the fd is registered]
+		[Instrument osmo_fd_register to check that the fd is registered]
 	)],
 	[fd_check=$enableval], [fd_check="no"])
 if test x"$fd_check" = x"no"
 then
-	AC_DEFINE([BSC_FD_CHECK],[1],[Instrument the bsc_register_fd])
+	AC_DEFINE([OSMO_FD_CHECK],[1],[Instrument the osmo_fd_register])
 fi
 
 AC_ARG_ENABLE(msgfile,
diff --git a/src/select.c b/src/select.c
index 774056a..496beea 100644
--- a/src/select.c
+++ b/src/select.c
@@ -121,7 +121,7 @@
 	if (fd->fd > maxfd)
 		maxfd = fd->fd;
 
-#ifdef BSC_FD_CHECK
+#ifdef OSMO_FD_CHECK
 	if (osmo_fd_is_registered(fd)) {
 		fprintf(stderr, "Adding a osmo_fd that is already in the list.\n");
 		return 0;

-- 
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/18169
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I053c2bfe461aa82085e7dac1cdcc95dd77219949
Gerrit-Change-Number: 18169
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <axilirator at gmail.com>
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: osmith <osmith at sysmocom.de>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200511/f420e410/attachment.htm>


More information about the gerrit-log mailing list