Attention is currently required from: laforge, osmith.
Hoernchen has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42330?usp=email )
Change subject: fw source: force reformat once and for all ......................................................................
Patch Set 1:
(1 comment)
File ccid_common/iso7816_fsm.c:
https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42330/comment/3c27f976_577... : PS1, Line 818: .in_event_mask = S(ISO7816_E_RX_SINGLE) | S(ISO7816_E_RX_COMPL) | S(ISO7816_E_TX_COMPL) | S(ISO7816_E_RX_ERR_IND) | S(ISO7816_E_TX_ERR_IND) | S(ISO7816_E_TPDU_DONE_IND),
I also don't consider this an improvement inreadability; it also goes far beyond the 120 chasr / lin […]
well that is in part easy to explain: https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42227 there is apparently no 120 limit anymore because someone turned off a month ago.. i suppose a more fine grained limit makes sense.