osmith has posted comments on this change. ( https://gerrit.osmocom.org/10216 )

Change subject: ipa_ccm_idtag_parse*: Fix reported length value
......................................................................


Patch Set 3: Code-Review+2

> Sure, but if I understood it correctly it's not related to commit itself so 
> it shouldn't stop us from merging this.

Then go ahead and merge it, you already have 2x +1 :)


--
To view, visit https://gerrit.osmocom.org/10216
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I435aaa33605bd48635715a2c81aa2d231c1abf51
Gerrit-Change-Number: 10216
Gerrit-PatchSet: 3
Gerrit-Owner: Harald Welte <[email protected]>
Gerrit-Assignee: Neels Hofmeyr <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Holger Freyther <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Max <[email protected]>
Gerrit-Reviewer: Neels Hofmeyr <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Comment-Date: Thu, 21 Feb 2019 14:44:39 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes

Reply via email to