Patch Set 2: Code-Review-1 (1 comment)
https://gerrit.osmocom.org/#/c/4645/2/src/host/layer23/src/mobile/gsm322.c File src/host/layer23/src/mobile/gsm322.c: Line 326: for (i = 0; i < value && i < sizeof(bar) - 1; i++) > you could just as well simply compute the length of the interval as a posit The main idea of this commit was to avoid a compiler warning related to possibility of calling memset with zero length argument, but seems I went a long way ;) I just found out a bit different way of solving this issue, so this change will be updated soon. -- To view, visit https://gerrit.osmocom.org/4645 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2d8d78474614939659a7f24d5007b1c890776b1a Gerrit-PatchSet: 2 Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Owner: Vadim Yanitskiy <axilira...@gmail.com> Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Max <msur...@sysmocom.de> Gerrit-Reviewer: Vadim Yanitskiy <axilira...@gmail.com> Gerrit-HasComments: Yes
