Change in osmo-msc[master]: use __FILE__, not __BASE_FILE__

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

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Mon Aug 20 11:08:12 UTC 2018


Neels Hofmeyr has uploaded this change for review. ( https://gerrit.osmocom.org/10539


Change subject: use __FILE__, not __BASE_FILE__
......................................................................

use __FILE__, not __BASE_FILE__

The intention was to use the file's basename, but __BASE_FILE__ means "the root
file that is being parsed and contains #include statements".

If we had a function using __BASE_FILE__ and that was defined in an #included
file, __BASE_FILE__ would indicate the first file where the #include is, and
not the file where the function is defined. __BASE_FILE__ works for us because
we don't ever include function definitions that log something, so __BASE_FILE__
always coincides with __FILE__ for our logging; but still __BASE_FILE__ is
semantically the wrong constant.

Related: OS#2740
Change-Id: I1c8122c909938daaf782468c1c5b0262d555c3ce
---
M include/osmocom/msc/osmo_msc.h
M include/osmocom/msc/vlr.h
2 files changed, 9 insertions(+), 9 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/39/10539/1

diff --git a/include/osmocom/msc/osmo_msc.h b/include/osmocom/msc/osmo_msc.h
index 995b195..7b3702a 100644
--- a/include/osmocom/msc/osmo_msc.h
+++ b/include/osmocom/msc/osmo_msc.h
@@ -101,9 +101,9 @@
 { return get_value_string(msc_subscr_conn_use_names, val); }
 
 #define msc_subscr_conn_get(conn, balance_token) \
-	_msc_subscr_conn_get(conn, balance_token, __BASE_FILE__, __LINE__)
+	_msc_subscr_conn_get(conn, balance_token, __FILE__, __LINE__)
 #define msc_subscr_conn_put(conn, balance_token) \
-	_msc_subscr_conn_put(conn, balance_token, __BASE_FILE__, __LINE__)
+	_msc_subscr_conn_put(conn, balance_token, __FILE__, __LINE__)
 struct gsm_subscriber_connection *
 _msc_subscr_conn_get(struct gsm_subscriber_connection *conn,
 		     enum msc_subscr_conn_use balance_token,
diff --git a/include/osmocom/msc/vlr.h b/include/osmocom/msc/vlr.h
index 91c28db..386a548 100644
--- a/include/osmocom/msc/vlr.h
+++ b/include/osmocom/msc/vlr.h
@@ -326,19 +326,19 @@
 const char *vlr_subscr_msisdn_or_name(struct vlr_subscr *vsub);
 
 #define vlr_subscr_find_by_imsi(vlr, imsi) \
-	_vlr_subscr_find_by_imsi(vlr, imsi, __BASE_FILE__, __LINE__)
+	_vlr_subscr_find_by_imsi(vlr, imsi, __FILE__, __LINE__)
 #define vlr_subscr_find_or_create_by_imsi(vlr, imsi, created) \
 	_vlr_subscr_find_or_create_by_imsi(vlr, imsi, created, \
-					   __BASE_FILE__, __LINE__)
+					   __FILE__, __LINE__)
 
 #define vlr_subscr_find_by_tmsi(vlr, tmsi) \
-	_vlr_subscr_find_by_tmsi(vlr, tmsi, __BASE_FILE__, __LINE__)
+	_vlr_subscr_find_by_tmsi(vlr, tmsi, __FILE__, __LINE__)
 #define vlr_subscr_find_or_create_by_tmsi(vlr, tmsi, created) \
 	_vlr_subscr_find_or_create_by_tmsi(vlr, tmsi, created, \
-					   __BASE_FILE__, __LINE__)
+					   __FILE__, __LINE__)
 
 #define vlr_subscr_find_by_msisdn(vlr, msisdn) \
-	_vlr_subscr_find_by_msisdn(vlr, msisdn, __BASE_FILE__, __LINE__)
+	_vlr_subscr_find_by_msisdn(vlr, msisdn, __FILE__, __LINE__)
 
 struct vlr_subscr *_vlr_subscr_find_by_imsi(struct vlr_instance *vlr,
 					    const char *imsi,
@@ -362,8 +362,8 @@
 					      const char *msisdn,
 					      const char *file, int line);
 
-#define vlr_subscr_get(sub) _vlr_subscr_get(sub, __BASE_FILE__, __LINE__)
-#define vlr_subscr_put(sub) _vlr_subscr_put(sub, __BASE_FILE__, __LINE__)
+#define vlr_subscr_get(sub) _vlr_subscr_get(sub, __FILE__, __LINE__)
+#define vlr_subscr_put(sub) _vlr_subscr_put(sub, __FILE__, __LINE__)
 struct vlr_subscr *_vlr_subscr_get(struct vlr_subscr *sub, const char *file, int line);
 struct vlr_subscr *_vlr_subscr_put(struct vlr_subscr *sub, const char *file, int line);
 

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

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I1c8122c909938daaf782468c1c5b0262d555c3ce
Gerrit-Change-Number: 10539
Gerrit-PatchSet: 1
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20180820/ce07bf87/attachment.htm>


More information about the gerrit-log mailing list