Attention is currently required from: Felix Singer, Thomas Heijligen.

View Change

1 comment:

• Patchset:

  • Patch Set #4:

    Hmmm, looks a bit too mixed, IMO. Would it be worth to split "edi"
    (i.e. the code on top up to edi_disable()) out into a separate file?

    Or maybe inline the definitions? After this patch things are interleaved:

    • EDI (definitions)
    • ENE (definitions)
    • EDI (code)
    • ENE (code)

    IMO, this would make things easier (if splitting into two files is a bad idea):

    • EDI (definitions, code)
    • ENE (definitions, code)

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