Attention is currently required from: pespin.

View Change

1 comment:

• File src/host/layer23/include/osmocom/bb/common/sysinfo.h:

  • Patch Set #2, Line 74: uint8_t supported;

    > Yes, but <stdbool.h> in not used this file. […]

    I do use bool wherever possible too. But here we're touching old code from 2010 and it's rather typical to see integers used to store boolean values. Not sure what exactly confuses you so much: 0 is false, anything else is true. As I said I prefer to stay consistent with the existing fields in this file. Feel free to submit patch(es) refactoring everything to use bool, if you have time.

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