Attention is currently required from: pespin.
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-cbc/+/28516 )

Change subject: Move header files to include/osmocom/cbc/ dir
......................................................................


Patch Set 2: Code-Review+1

(1 comment)

Patchset:

PS2:
would be interesting to state why (I guess for consistency with our other 
projects?)

I personally think they should actually all be in src and only public (library) 
headers should be in include/osmocom/* :P



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

Gerrit-Project: osmo-cbc
Gerrit-Branch: master
Gerrit-Change-Id: Id6c9f99805169d624a2c6e001fe1983e24fdc5c3
Gerrit-Change-Number: 28516
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Comment-Date: Mon, 04 Jul 2022 18:26:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Reply via email to