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
th
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: co
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 (#7).
tests/conv: add GSM 05.03 specific test
This change extends the convolutional code test coverage, adding
the GSM 05.03 specific tes
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 tes
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 tes
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 §.
The older 3GPP specification naming (05.03) is still used throughout -
filenames for example. Pe
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 https://gerrit.osmocom.org/set
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 0x
