Attention is currently required from: jolly.

View Change

3 comments:

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

  • Patch Set #1, Line 19: TransactionId_t *eim_transaction_id;

    As it seems there is a tab instead of a space between type and member name. The member name is consistent, however in other code locations we use struct OCTET_STRING to declare the type.

    (here the name of the struct meber is consistent)

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

  • Patch Set #1, Line 19: TransactionId_t *eim_transaction_id;

    maybe declare this as struct OCTET_STRING *eim_transaction_id?

    It could also make sense to apply some refactoring, so that we consistently use TransactionId_t throughout the entire codebase. What we should not do is to mix the two type names, even when they are the same under to hood.

    We may also check if it would work to declare the eim_transaction_id as const n the pars and req structs.

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

  • Patch Set #1, Line 18: TransactionId_t *eim_transaction_id;

    (see the other comments)

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