Change in osmo-bsc[master]: cosmetic: bsc: timeslot_fsm: Clean uneeded scope brackets

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

Pau Espin Pedrol gerrit-no-reply at lists.osmocom.org
Wed Nov 28 11:51:08 UTC 2018


Pau Espin Pedrol has submitted this change and it was merged. ( https://gerrit.osmocom.org/11955 )

Change subject: cosmetic: bsc: timeslot_fsm: Clean uneeded scope brackets
......................................................................

cosmetic: bsc: timeslot_fsm: Clean uneeded scope brackets

Change-Id: I5140f98e23b8c8d16ce0cca0be66297aaf0b5653
---
M src/osmo-bsc/timeslot_fsm.c
1 file changed, 4 insertions(+), 5 deletions(-)

Approvals:
  Neels Hofmeyr: Looks good to me, approved
  Vadim Yanitskiy: Looks good to me, but someone else must approve
  Jenkins Builder: Verified



diff --git a/src/osmo-bsc/timeslot_fsm.c b/src/osmo-bsc/timeslot_fsm.c
index fc25460..8351ff7 100644
--- a/src/osmo-bsc/timeslot_fsm.c
+++ b/src/osmo-bsc/timeslot_fsm.c
@@ -911,6 +911,8 @@
  * and will be served after the switch. (Do not check whether any lchans are actually available.) */
 bool ts_usable_as_pchan(struct gsm_bts_trx_ts *ts, enum gsm_phys_chan_config as_pchan)
 {
+	enum gsm_phys_chan_config target_pchan;
+
 	if (!ts_is_usable(ts))
 		return false;
 
@@ -922,11 +924,8 @@
 		break;
 	}
 
-	{
-		enum gsm_phys_chan_config target_pchan;
-		if (ts_is_lchan_waiting_for_pchan(ts, &target_pchan))
-			return target_pchan == as_pchan;
-	}
+	if (ts_is_lchan_waiting_for_pchan(ts, &target_pchan))
+		return target_pchan == as_pchan;
 
 	return ts_is_capable_of_pchan(ts, as_pchan);
 }

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I5140f98e23b8c8d16ce0cca0be66297aaf0b5653
Gerrit-Change-Number: 11955
Gerrit-PatchSet: 2
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20181128/29277dfe/attachment.htm>


More information about the gerrit-log mailing list