Change in osmo-mgw[master]: mgcp_client: drop nunnecessary else statement

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

dexter gerrit-no-reply at lists.osmocom.org
Mon Jun 7 20:40:45 UTC 2021


dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-mgw/+/24602 )


Change subject: mgcp_client: drop nunnecessary else statement
......................................................................

mgcp_client: drop nunnecessary else statement

There is an elarly return statement, so there is no need for an else
branch.

Change-Id: I96d0d468ccab302f9add206164f4d5b1b768bb48
---
M src/libosmo-mgcp-client/mgcp_client.c
1 file changed, 1 insertion(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/02/24602/1

diff --git a/src/libosmo-mgcp-client/mgcp_client.c b/src/libosmo-mgcp-client/mgcp_client.c
index fbc4cf5..7367b40 100644
--- a/src/libosmo-mgcp-client/mgcp_client.c
+++ b/src/libosmo-mgcp-client/mgcp_client.c
@@ -1390,8 +1390,7 @@
 		rc_sdp = add_sdp(msg, mgcp_msg, mgcp);
 		if (rc_sdp == -2)
 			return NULL;
-		else
-			rc += rc_sdp;
+		rc += rc_sdp;
 	}
 
 	if (rc != 0) {

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/24602
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: I96d0d468ccab302f9add206164f4d5b1b768bb48
Gerrit-Change-Number: 24602
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210607/a9806964/attachment.htm>


More information about the gerrit-log mailing list