Harald Welte has submitted this change and it was merged. Change subject: VLR tests: remove weird code ......................................................................
VLR tests: remove weird code Having while(0) as a body for for() cycle generate too much WTF while looking at the code while not serving any obvious purpose. Let's just drop it to avoid confusing developers. Change-Id: I1f571ec319ff3231fd9acd4066e470476c3b1f78 Related: OS#2864 --- M tests/msc_vlr/msc_vlr_tests.c 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/tests/msc_vlr/msc_vlr_tests.c b/tests/msc_vlr/msc_vlr_tests.c index f373f1e..c639f98 100644 --- a/tests/msc_vlr/msc_vlr_tests.c +++ b/tests/msc_vlr/msc_vlr_tests.c @@ -847,7 +847,7 @@ fprintf(stderr, "(test nr %d)\n", test_nr + 1); check_talloc(msgb_ctx, tall_bsc_ctx, 8); - } while(0); + } } int main(int argc, char **argv) -- To view, visit https://gerrit.osmocom.org/6305 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1f571ec319ff3231fd9acd4066e470476c3b1f78 Gerrit-PatchSet: 4 Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Owner: Max <msur...@sysmocom.de> Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillm...@sysmocom.de> Gerrit-Reviewer: neels <nhofm...@sysmocom.de>