Attention is currently required from: pespin.

View Change

8 comments:

• File src/ss7_asp_vty.c:

  • Patch Set #1, Line 423: asp->cfg.ip_dscp = 0;

    Ok fine, but since here you are resetting ip_dscp, I think it's a good idea to if (asp->cfg. […]

    Done

• File src/ss7_asp_vty.c:

  • Patch Set #3, Line 440: ss7_asp_apply_ip_dscp(asp);

    btw, it may make sense to rename this "ss7_asp_apply_ip_dscp" into "ss7_asp_apply_qos_class()"

    Done

  • Patch Set #3, Line 467:

    Better keep the qos-class commands in this place rather than above, to follow the order in ss7_vty_i […]

    Done

  • Patch Set #3, Line 1458: if (asp->cfg.ip_dscp != 0)

    you can use "else if" here, to denote explicitly that ip-dscp can only be >0 if qos_class is not set […]

    Done

• File src/ss7_xua_srv.c:

  • Patch Set #3, Line 417: else

    logic changed here, missing "if (xs->cfg.ip_dscp != 0)". […]

    Done

• File src/ss7_xua_srv_vty.c:

  • Patch Set #3, Line 152: DEFUN_ATTR(xua_qos_class, xua_qos_class_cmd,

    Move these further below to match ss7_vty_init_node_oxs() order.

    Done

  • Patch Set #3, Line 289: if (xs->cfg.ip_dscp)

    else if

    Done

  • Patch Set #3, Line 407: install_lib_element(L_CS7_XUA_NODE, &xua_qos_class_cmd);

    Move this below init_ip_dscp, as in the asp vty case.

    Done

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