Attention is currently required from: laforge, lynxis lazus.

Patch set 1:Code-Review +1

View Change

2 comments:

• Patchset:

  • Patch Set #1:

    To me this looks ok and apparently it was also tested in practice.

• Commit Message:

  • Patch Set #1, Line 28: the future.

    I think having unittests would be very helpful. This would also make reviewing easier. At the moment I find it a bit hard to follow those SAIP related patches.

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