Attention is currently required from: osmith, fixeria.

View Change

1 comment:

• File src/gprs/gprs_llc_parse.c:

  • Patch Set #1, Line 138: ghp->data = (uint8_t *)&ctrl[3];

    Yes, but why doing this is wrong? gprs_llc_hdr_parse() itself does not modify *llc_hdr, so it's cons […]

    The compiler may make optimizations based on what passed to the function. If a const pointer is passed to a function, that means it can be inferred that the content of array remains invariable regardless of whether the function was called or not. You are actually telling the compiler one thing while actually sneakily changing what you announced. That's why casting should be done carefully and with a good reason.
    If the data content being passed here will need to be writeable, bear with it and don't announce it's const.

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