Patch Set 1: Code-Review-1 (2 comments)
https://gerrit.osmocom.org/#/c/2876/1/src/viterbi.c File src/viterbi.c: Line 378: vdec_free(trellis->sums); I would rather change vdec_free() to accept NULL as a valid argument. Keeps the code cleaner. Line 381: free(trellis->vals); free() on NULL is perfectly legal -- To view, visit https://gerrit.osmocom.org/2876 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I36012d4443d97470050cdf9638a9d4cf67ea3b40 Gerrit-PatchSet: 1 Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Owner: Vadim Yanitskiy <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-HasComments: Yes
