Attention is currently required from: pespin.

arehbein has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/31303 )

Change subject: Includes: Add gprs/rlc related enums
......................................................................


Patch Set 7:

(1 comment)

Patchset:

PS7:
> why are these still kept here? I think we already agreed that those are not 
> really enums specified i […]
My bad, seems like I wrote a response but kept in in draft mode without sending 
the replies.

For now I'm still keeping them there so I can keep working on the other parts 
of this (since updating the enum names/includes doesn't affect the rest of this 
problem much) and I don't yet have a better idea of where to put it. You have a 
reply in your first comment regarding this (please also see what I wrote in the 
related issue on Redmine)



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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I1ec2e52ad7fa625c08fe9df04d4a4f5323bf609c
Gerrit-Change-Number: 31303
Gerrit-PatchSet: 7
Gerrit-Owner: arehbein <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-CC: laforge <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Comment-Date: Mon, 20 Feb 2023 14:07:48 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <[email protected]>
Gerrit-MessageType: comment

Reply via email to