Change in libosmocore[master]: logging: Remove duplicate color for DLSMS
laforge has submitted this change. (
https://gerrit.osmocom.org/c/libosmocore/+/22027 )
Change subject: logging: Remove duplicate color for DLSMS
..
logging: Remove duplicate color for DLSMS
Change-Id: Ia797efce36e0afff74de54374d568521ecedf486
---
M src/logging.c
1 file changed, 0 insertions(+), 1 deletion(-)
Approvals:
laforge: Looks good to me, approved
pespin: Looks good to me, but someone else must approve
Jenkins Builder: Verified
diff --git a/src/logging.c b/src/logging.c
index 01a00da..60dba64 100644
--- a/src/logging.c
+++ b/src/logging.c
@@ -177,7 +177,6 @@
.name = "DLSMS",
.description = "Layer3 Short Message Service (SMS)",
.enabled = 1, .loglevel = LOGL_NOTICE,
- .color = OSMO_LOGCOLOR_BRIGHTWHITE,
.color = "\033[38;5;31m",
},
[INT2IDX(DLCTRL)] = {
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/22027
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ia797efce36e0afff74de54374d568521ecedf486
Gerrit-Change-Number: 22027
Gerrit-PatchSet: 2
Gerrit-Owner: daniel
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge
Gerrit-Reviewer: pespin
Gerrit-MessageType: merged
Change in libosmocore[master]: logging: Remove duplicate color for DLSMS
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/22027 ) Change subject: logging: Remove duplicate color for DLSMS .. Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/22027 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Ia797efce36e0afff74de54374d568521ecedf486 Gerrit-Change-Number: 22027 Gerrit-PatchSet: 1 Gerrit-Owner: daniel Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge Gerrit-Reviewer: pespin Gerrit-Comment-Date: Thu, 07 Jan 2021 18:57:16 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in libosmocore[master]: logging: Remove duplicate color for DLSMS
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/22027 ) Change subject: logging: Remove duplicate color for DLSMS .. Patch Set 1: Code-Review+1 Would maybe make sense remove the non-defined string. -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/22027 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Ia797efce36e0afff74de54374d568521ecedf486 Gerrit-Change-Number: 22027 Gerrit-PatchSet: 1 Gerrit-Owner: daniel Gerrit-Reviewer: pespin Gerrit-CC: Jenkins Builder Gerrit-Comment-Date: Thu, 07 Jan 2021 14:01:17 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in libosmocore[master]: logging: Remove duplicate color for DLSMS
daniel has uploaded this change for review. (
https://gerrit.osmocom.org/c/libosmocore/+/22027 )
Change subject: logging: Remove duplicate color for DLSMS
..
logging: Remove duplicate color for DLSMS
Change-Id: Ia797efce36e0afff74de54374d568521ecedf486
---
M src/logging.c
1 file changed, 0 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/27/22027/1
diff --git a/src/logging.c b/src/logging.c
index 01a00da..60dba64 100644
--- a/src/logging.c
+++ b/src/logging.c
@@ -177,7 +177,6 @@
.name = "DLSMS",
.description = "Layer3 Short Message Service (SMS)",
.enabled = 1, .loglevel = LOGL_NOTICE,
- .color = OSMO_LOGCOLOR_BRIGHTWHITE,
.color = "\033[38;5;31m",
},
[INT2IDX(DLCTRL)] = {
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/22027
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ia797efce36e0afff74de54374d568521ecedf486
Gerrit-Change-Number: 22027
Gerrit-PatchSet: 1
Gerrit-Owner: daniel
Gerrit-MessageType: newchange
