Attention is currently required from: fixeria.

View Change

1 comment:

• Patchset:

  • Patch Set #2:

    So far I had a quick look and I don't really understand/see what do we benefit from this refactoring.

    As mentioned in the commit message, these params follow the same lifecycle than that the state, ie. they get per-lchan modified over RSL. By having it inside the struct, we reset the whole state together, so the logic is simplified.

    And using the pointer allows to avoid memcpy()ing parameters which are identical for all lchans in 99% cases.

    You are still holding a copy on each lchan. All I'm doing is getting rid of one extra pointer field.

     I don't see how this pointer being NULL can crate problems in static mode: one shall simply never need to access dynamic power control parameters in static mode.

    Yes, that's usually how crashes happen: One ends up doing something should never need to do 😊

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