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.orgdexter has submitted this change. ( 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(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved laforge: Looks good to me, approved diff --git a/src/libosmo-mgcp-client/mgcp_client.c b/src/libosmo-mgcp-client/mgcp_client.c index a134273..55ce7b2 100644 --- a/src/libosmo-mgcp-client/mgcp_client.c +++ b/src/libosmo-mgcp-client/mgcp_client.c @@ -1378,8 +1378,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: 2 Gerrit-Owner: dexter <pmaier at sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier at sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy at sysmocom.de> Gerrit-Reviewer: laforge <laforge at osmocom.org> Gerrit-Reviewer: pespin <pespin at sysmocom.de> Gerrit-MessageType: merged -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210611/b6698b31/attachment.htm>