Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/29694 )
Change subject: msc_a,vlr: rename ciphering_required to try_ciphering ......................................................................
Patch Set 1:
(6 comments)
File src/libvlr/vlr_access_req_fsm.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-42): https://gerrit.osmocom.org/c/osmo-msc/+/29694/comment/733d4069_d92429f6 PS1, Line 668: (authentication_required || try_ciphering)? spaces required around that '?' (ctx:VxE)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-42): https://gerrit.osmocom.org/c/osmo-msc/+/29694/comment/746319a7_b0823302 PS1, Line 670: (authentication_required || try_ciphering)? spaces required around that '?' (ctx:VxE)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-42): https://gerrit.osmocom.org/c/osmo-msc/+/29694/comment/b5f666d8_75e05e82 PS1, Line 671: (try_ciphering? "+Ciph" : " (no Ciph)") spaces required around that '?' (ctx:VxW)
File src/libvlr/vlr_lu_fsm.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-42): https://gerrit.osmocom.org/c/osmo-msc/+/29694/comment/998c974e_c8ce652b PS1, Line 1517: (authentication_required || try_ciphering)? spaces required around that '?' (ctx:VxE)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-42): https://gerrit.osmocom.org/c/osmo-msc/+/29694/comment/fb4c3118_72b169c6 PS1, Line 1519: (authentication_required || try_ciphering)? spaces required around that '?' (ctx:VxE)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-42): https://gerrit.osmocom.org/c/osmo-msc/+/29694/comment/fbfd6259_91937152 PS1, Line 1520: (try_ciphering? "+Ciph" : " (no Ciph)") spaces required around that '?' (ctx:VxW)
-
Jenkins Builder