laforge has submitted this change. (
https://gerrit.osmocom.org/c/osmo-e1d/+/27584 )
Change subject: cosmetic: curly braces around multi-line blocks
......................................................................
cosmetic: curly braces around multi-line blocks
This reflects our general coding style.
Change-Id: Ibf8ebe4c9fdde2bd66ce6239c8eca833e71fc815
---
M src/intf_line.c
1 file changed, 4 insertions(+), 2 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
tnt: Looks good to me, but someone else must approve
diff --git a/src/intf_line.c b/src/intf_line.c
index c9011a8..9146368 100644
--- a/src/intf_line.c
+++ b/src/intf_line.c
@@ -103,9 +103,10 @@
if (!serial_str)
return NULL;
- llist_for_each_entry(intf, &e1d->interfaces, list)
+ llist_for_each_entry(intf, &e1d->interfaces, list) {
if (intf->usb.serial_str && !strcmp(intf->usb.serial_str, serial_str))
return intf;
+ }
return NULL;
}
@@ -115,9 +116,10 @@
{
struct e1_line *line;
- llist_for_each_entry(line, &intf->lines, list)
+ llist_for_each_entry(line, &intf->lines, list) {
if (line->id == id)
return line;
+ }
return NULL;
}
1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-e1d/+/27584
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-e1d
Gerrit-Branch: master
Gerrit-Change-Id: Ibf8ebe4c9fdde2bd66ce6239c8eca833e71fc815
Gerrit-Change-Number: 27584
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: tnt <tnt(a)246tNt.com>
Gerrit-MessageType: merged