Attention is currently required from: laforge, pespin.

Patch set 8:Code-Review +1

View Change

1 comment:

• File src/ipa.c:

  • Patch Set #3, Line 50: #i

    I tend to agree, but given that the previous state was already quite caothic with files already name […]

    I'm not sure I understand the argument. Why not name the new file ss7_asp_internal.h instead? I think that would make sense even if we already have other internal header files without _internal suffix.

    I'd slightly prefer that, otherwise LGTM.

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