Change in ...libosmocore[master]: utils.h: require a semi colon after OSMO_ASSERT

lynxis lazus gerrit-no-reply at lists.osmocom.org
Sun Jun 23 00:08:33 UTC 2019


Hello Jenkins Builder, 

I'd like you to reexamine a change. Please visit

    https://gerrit.osmocom.org/c/libosmocore/+/14578

to look at the new patch set (#3).

Change subject: utils.h: require a semi colon after OSMO_ASSERT
......................................................................

utils.h: require a semi colon after OSMO_ASSERT

When using `OSMO_ASSERT(exp);` clang will warn about
an empty expression because the semi colon was superflous.
Use do {} while (0) to enfore the need of a semi colon.
This might break other test.

Change-Id: I2272d29a81496164bebd1696a694383a28a86434
---
M include/osmocom/core/utils.h
M src/gsm/gsm0808_utils.c
M tests/ctrl/ctrl_test.c
3 files changed, 6 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/78/14578/3
-- 
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14578
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I2272d29a81496164bebd1696a694383a28a86434
Gerrit-Change-Number: 14578
Gerrit-PatchSet: 3
Gerrit-Owner: lynxis lazus <lynxis at fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190623/a5ed10b5/attachment.html>


More information about the gerrit-log mailing list