Attention is currently required from: pespin.

View Change

7 comments:

• File src/osmux_input.c:

  • Robot Comment from checkpatch (run ID jenkins-gerrit-libosmo-netif-lint-128):

    Patch Set #2, Line 181: osmuxh->amr_q= 0;

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

  • Robot Comment from checkpatch (run ID jenkins-gerrit-libosmo-netif-lint-128):

    Patch Set #2, Line 373: for (i=1; i<diff; i++) {

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

  • Robot Comment from checkpatch (run ID jenkins-gerrit-libosmo-netif-lint-128):

    Patch Set #2, Line 373: for (i=1; i<diff; i++) {

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

  • Robot Comment from checkpatch (run ID jenkins-gerrit-libosmo-netif-lint-128):

    Patch Set #2, Line 559: switch(rtph->payload_type) {

    switch and case should be at the same indent

  • Robot Comment from checkpatch (run ID jenkins-gerrit-libosmo-netif-lint-128):

    Patch Set #2, Line 559: switch(rtph->payload_type) {

    space required before the open parenthesis '('

• File src/osmux_output.c:

  • Robot Comment from checkpatch (run ID jenkins-gerrit-libosmo-netif-lint-128):

    Patch Set #2, Line 206: * them for transmission at appropiate time.

    'appropiate' may be misspelled - perhaps 'appropriate'?

  • Robot Comment from checkpatch (run ID jenkins-gerrit-libosmo-netif-lint-128):

    Patch Set #2, Line 255: if(was_empty && !llist_empty(&h->list))

    space required before the open parenthesis '('

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