Attention is currently required from: laforge, fixeria, pespin.

View Change

1 comment:

• File tests/osmo_io/osmo_io_test.c:

  • Patch Set #9, Line 95: for (int i = 0; i < 128; i++)

    Huh? Not sure about "we". I think we don't since we bumped the `-std` to `gnu11`. […]

    I would like to keep those as is tbh. I was happy once we moved to a c std that supports those and I think it has other advantages as not cluttering the surrounding scope, you can't forget to remove the variable if you remove the loop.

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