Vadim Yanitskiy has submitted this change and it was merged.
Change subject: tests/conv: add GSM 05.03 specific test
..
tests/conv: add GSM 05.03 specific test
This change extends the convolutional code test coverage, adding
Patch Set 7: Code-Review+2
deferring to Tom's judgement here, but generally it looks fine to me, and if
there are issues remaining, they can be adressed later on.
--
To view, visit https://gerrit.osmocom.org/1628
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType:
Hello Max, Harald Welte, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/1628
to look at the new patch set (#6).
tests/conv: add GSM 05.03 specific test
This change extends the convolutional code test coverage, adding
the GSM 05.03 specific
Hello Max, Harald Welte, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/1628
to look at the new patch set (#5).
tests/conv: add GSM 05.03 specific test
This change extends the convolutional code test coverage, adding
the GSM 05.03 specific
Patch Set 4:
I think we should refer to latest version of the standard in here which is 3GPP
TS 45.003 and if the test vectors are part of the spec than we should even
refer to particular ยง.
--
To view, visit https://gerrit.osmocom.org/1628
To unsubscribe, visit
Patch Set 4:
the address sanitizer bug will have to be resolved first, befor the patch is
verified and can be merged:
+=
+==23280==ERROR: AddressSanitizer: heap-buffer-overflow on address
0x6150d700 at pc 0x7f490faf014e bp