Change in libosmocore[master]: gsm0502: use parentheses in GSM_TDMA_FN_{SUM,SUB} macros

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmocore/+/29579 ) Change subject: gsm0502: use parentheses in GSM_TDMA_FN_{SUM,SUB} macros ...................................................................... gsm0502: use parentheses in GSM_TDMA_FN_{SUM,SUB} macros This makes it possible to pass expressions to the abovementioned macros without any side-effects. Strictly speaking, this is only necessary for argument 'b' of GSM_TDMA_FN_SUB, but let's also add them to GSM_TDMA_FN_SUM for consistency. Change-Id: I7f26156813139bbb7774e1eb342c91bc84fdad38 --- M include/osmocom/gsm/gsm0502.h 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/79/29579/1 diff --git a/include/osmocom/gsm/gsm0502.h b/include/osmocom/gsm/gsm0502.h index 1be2cc3..b6a1807 100644 --- a/include/osmocom/gsm/gsm0502.h +++ b/include/osmocom/gsm/gsm0502.h @@ -16,10 +16,10 @@ /*! Return the sum of two specified TDMA frame numbers (summation) */ #define GSM_TDMA_FN_SUM(a, b) \ - ((a + b) % GSM_TDMA_HYPERFRAME) + (((a) + (b)) % GSM_TDMA_HYPERFRAME) /*! Return the difference of two specified TDMA frame numbers (subtraction) */ #define GSM_TDMA_FN_SUB(a, b) \ - ((a + GSM_TDMA_HYPERFRAME - b) % GSM_TDMA_HYPERFRAME) + (((a) + GSM_TDMA_HYPERFRAME - (b)) % GSM_TDMA_HYPERFRAME) /*! Return the *minimum* difference of two specified TDMA frame numbers (distance) */ #define GSM_TDMA_FN_DIFF(a, b) \ OSMO_MIN(GSM_TDMA_FN_SUB(a, b), GSM_TDMA_FN_SUB(b, a)) -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/29579 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I7f26156813139bbb7774e1eb342c91bc84fdad38 Gerrit-Change-Number: 29579 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange