Attention is currently required from: pespin.

View Change

1 comment:

• Patchset:

  • Patch Set #1:

    IMHO you shouldn't drop the arg. […]

    I'm not dropping it from ranap_cn_rx_co and ranap_cn_rx_cl, but only from functions that these are calling in the same file. Because these functions don't use ctx at all, so imho having it there just makes the whole code more confusing as it's not clear whether ctx is for userdata or a talloc context when looking only at parts of it.

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