View Change

6 comments:

• File src/osmo-hnbgw/hnbgw_vty.c:

  • Robot Comment from checkpatch (run ID jenkins-gerrit-lint-7702):

    Patch Set #1, Line 357: #define NRI_ARGS_TO_STR_ARGS(ARGC, ARGV) ARGV[0], (ARGC>1)? ".." : "", (ARGC>1)? ARGV[1] : ""

    spaces required around that '>' (ctx:VxV)

  • Robot Comment from checkpatch (run ID jenkins-gerrit-lint-7702):

    Patch Set #1, Line 357: #define NRI_ARGS_TO_STR_ARGS(ARGC, ARGV) ARGV[0], (ARGC>1)? ".." : "", (ARGC>1)? ARGV[1] : ""

    spaces required around that '?' (ctx:VxW)

  • Robot Comment from checkpatch (run ID jenkins-gerrit-lint-7702):

    Patch Set #1, Line 357: #define NRI_ARGS_TO_STR_ARGS(ARGC, ARGV) ARGV[0], (ARGC>1)? ".." : "", (ARGC>1)? ARGV[1] : ""

    spaces required around that '>' (ctx:VxV)

  • Robot Comment from checkpatch (run ID jenkins-gerrit-lint-7702):

    Patch Set #1, Line 357: #define NRI_ARGS_TO_STR_ARGS(ARGC, ARGV) ARGV[0], (ARGC>1)? ".." : "", (ARGC>1)? ARGV[1] : ""

    spaces required around that '?' (ctx:VxW)

  • Robot Comment from checkpatch (run ID jenkins-gerrit-lint-7702):

    Patch Set #1, Line 518: if (message) {

    braces {} are not necessary for single statement blocks

  • Robot Comment from checkpatch (run ID jenkins-gerrit-lint-7702):

    Patch Set #1, Line 558: if (message) {

    braces {} are not necessary for single statement blocks

To view, visit change 33130. To unsubscribe, or for help writing mail filters, visit settings.