Attention is currently required from: wbokslag.

Patch set 2:Code-Review +1

View Change

2 comments:

• Patchset:

  • Patch Set #2:

    Not really critical, looks good in general.

• File src/tetra_llc_pdu.h:

  • Patch Set #1, Line 80:

    tl_sdu_len needs to be changed from a uint8_t to uint32_t because, due to defragmentation, pdus may […]

    I meant to say "unrelated WhiteSpace change", sorry for not being clear. What I wanted to point out is that you're adding two tabs here for no reason. You're changing the line below, this one should remain unchanged.

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