Max has posted comments on this change. ( https://gerrit.osmocom.org/11826 )

Change subject: LCLS: add gsm0808_create_ass_ext()
......................................................................


Patch Set 7:

> Is it a different message from the existing function?

It's the same, I've added spec. ref. to clarify.

> Isn't ci inside lcls structure?

It's not. Moreover, from my understanding of the spec it's not related to LCLS 
- I've updated confusing comment.

The rest should be fixed in latest revision hence removed WIP mark.


--
To view, visit https://gerrit.osmocom.org/11826
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: Id89765df3f8c12f55f73f1d7a9d90c8883eb3bba
Gerrit-Change-Number: 11826
Gerrit-PatchSet: 7
Gerrit-Owner: Max <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Max <[email protected]>
Gerrit-CC: Pau Espin Pedrol <[email protected]>
Gerrit-Comment-Date: Mon, 19 Nov 2018 12:30:32 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No

Reply via email to