Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/30933 )
Change subject: uitils: add floored and euclidian modulo functions ...................................................................... Patch Set 2: (3 comments) File tests/utils/utils_test.c: Robot Comment from checkpatch (run ID jenkins-gerrit-lint-2601): https://gerrit.osmocom.org/c/libosmocore/+/30933/comment/744f3f15_73eb9b4e PS2, Line 773: printf(" %i mod %i = %i = %i\n", x, y, result, expected_result); Use %d instead of %i Robot Comment from checkpatch (run ID jenkins-gerrit-lint-2601): https://gerrit.osmocom.org/c/libosmocore/+/30933/comment/df39132f_57e5c72d PS2, Line 781: printf(" %i mod_flr %i = %i = %i\n", x, y, result, expected_result); Use %d instead of %i Robot Comment from checkpatch (run ID jenkins-gerrit-lint-2601): https://gerrit.osmocom.org/c/libosmocore/+/30933/comment/919cc5df_450a985c PS2, Line 789: printf(" %i mod_euc %i = %i = %i\n", x, y, result, expected_result); Use %d instead of %i -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/30933 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: If61cd54f43643325c45f64531c57fe4c5802a9cf Gerrit-Change-Number: 30933 Gerrit-PatchSet: 2 Gerrit-Owner: dexter <[email protected]> Gerrit-CC: Jenkins Builder Gerrit-Comment-Date: Wed, 11 Jan 2023 12:18:26 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
