laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/14711 )
Change subject: add semi-color after OSMO_ASSERT ...................................................................... Patch Set 1: Code-Review+1 (1 comment) https://gerrit.osmocom.org/#/c/14711/1//COMMIT_MSG Commit Message: https://gerrit.osmocom.org/#/c/14711/1//COMMIT_MSG@9 PS1, Line 9: In preparation of changing the OSMO_ASSERT() to require a semicolon. it's good to fix this here. however, we cannot change OSMO_ASSERT as we must ensure newer libosmocore versions work with older applications. -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/14711 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: I42fca73393d14ebed973616b86fb87f89bdbf095 Gerrit-Change-Number: 14711 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <[email protected]> Gerrit-Comment-Date: Wed, 10 Jul 2019 01:02:43 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
