[MERGED] osmocom-bb[master]: mobile: Use new LOGPSRCC macro to print multiple values

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

Holger Freyther gerrit-no-reply at lists.osmocom.org
Sun Dec 3 18:09:43 UTC 2017


Holger Freyther has submitted this change and it was merged.

Change subject: mobile: Use new LOGPSRCC macro to print multiple values
......................................................................


mobile: Use new LOGPSRCC macro to print multiple values

We need continuation to avoid printing the logging category
again. E.g. when print(one, two, three) is called.

Change-Id: Id8491fa949768f170a8c74ab554cb1166afda1b7
---
M src/host/layer23/src/mobile/script_lua.c
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/host/layer23/src/mobile/script_lua.c b/src/host/layer23/src/mobile/script_lua.c
index cbba070..422f9e4 100644
--- a/src/host/layer23/src/mobile/script_lua.c
+++ b/src/host/layer23/src/mobile/script_lua.c
@@ -56,10 +56,11 @@
 	lua_getstack(L, 1, &ar);
 	lua_getinfo(L, "nSl", &ar);
 
+	LOGPSRC(DLUA, loglevel, ar.source, ar.currentline, "%s", "");
 	for (i = 1; i <= argc; ++i) {
 		if (!lua_isstring(L, i))
 			continue;
-		LOGPSRC(DLUA, loglevel, ar.source, ar.currentline,
+		LOGPSRCC(DLUA, loglevel, ar.source, ar.currentline, 1,
 				"%s%s", i > 1 ? "\t" : "", lua_tostring(L, i));
 	}
 	LOGPC(DLUA, loglevel, "\n");

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id8491fa949768f170a8c74ab554cb1166afda1b7
Gerrit-PatchSet: 3
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Owner: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Jenkins Builder



More information about the gerrit-log mailing list