Attention is currently required from: fixeria, pespin.

View Change

6 comments:

• File Transceiver52M/grgsm_vitac/grgsm_vitac.h:

  • Patch Set #6, Line 61: MULTI_VER_TARGET_ATTR_CLANGONLY

    Why removing this? If on purpose, please explain in the commit message.

    Done

  • Patch Set #6, Line 62: detect_burst_access_burst

    `detect_burst_ab()` for consistency with `detect_burst_nb()` then?

    Done

• File Transceiver52M/grgsm_vitac/grgsm_vitac.cpp:

  • Patch Set #6, Line 77: MULTI_VER_TARGET_ATTR NO_UBSAN

    Why removing this? If on purpose, please explain in the commit message.

    Done

  • Patch Set #6, Line 42: d_sync_training_seq[N_ACCESS_BITS]; ///<encoded training sequence of a SCH burst

    I find this confusing: the comment mentions SCH, the symbol name contains `_sync_`, and the `N_ACCES […]

    Done

  • Patch Set #6, Line 71: gr_complex startpoint = train_seq[i][0] == 0 ? gr_complex(1.0, 0.0) : gr_complex(-1.0, 0.0);

    Unrelated spacing changes. […]

    This file was mostly unformatted before to keep it "in sync" with gr-gsm, but now, due to all those changes, trying to keep the original formatting is pointless, which is why it now got automatically reformatted with the osmo-trx (kernel) coding style, which has longer line limits.

  • Patch Set #6, Line 139: gmsk_output[i] = j * gr_complex(encoded_symbol, 0.0) * gmsk_output[i - 1];

    Again, I am fine with proper formatting, but it makes your patch harder to read.

    Done

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