Attention is currently required from: iedemam, neels.

View Change

1 comment:

• File src/osmo-bsc/lchan_fsm.c:

  • Patch Set #3, Line 519: .active_start = { .tv_sec = 0, .tv_nsec = 0 },

    The purpose of this was to initialize the timestamps to zero values so I can tell when they've been […]

    My point is simply that this struct initializer is already zeroing all fields in the struct unless otherwise specified, that's why most of the struct fields don't appear here.
    So there's no need to set them to 0 explicitly, they are set to zero anyway, so they can be removed.

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