Attention is currently required from: jolly.

Patch set 3:Code-Review +1

View Change

1 comment:

• File src/ipa/libipa/es10b_exec_fallback_mech.h:

  • Patch Set #3, Line 14: struct ExecuteFallbackMechanismResponse *res;

    The response is basically just an integer, you could return that integer from ipa_es10b_exec_fallback_mech directly. This would allow you to get rid of ipa_es10b_exec_fallback_mech_res_free and the API would me much simpler.

    However, the downside is that there would be less flexibility in case the response struct gets extended.

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