Attention is currently required from: fixeria, laforge, osmith.

View Change

1 comment:

• File src/ss7_asp.c:

  • Patch Set #3, Line 801:

    mh. […]

    This is already used in several places in our code base, check with:

        $ ag "^\s*\(void\)" libosmo* osmo-* | wc -l
        141

    This is also allowed just fine by the C specs:
    https://stackoverflow.com/questions/79100782/compiler-complains-that-an-unused-variable-is-used-when-simply-a-cast-to-void-is

    I'd prefer keeping this well-established way which is also useful in general for all compilers instead of sticking to something coverity specific.

    If you don't agree, I can remove the rc value completely and be done with it.

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