Attention is currently required from: pespin, fixeria.

arehbein has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/32354 )

Change subject: gsm_objclass2obj(): Change signature/set NACK cause
......................................................................


Patch Set 3:

(2 comments)

File src/common/oml.c:

https://gerrit.osmocom.org/c/osmo-bts/+/32354/comment/2a078191_80ec2266
PS2, Line 537:  enum abis_nm_nack_cause c;
> This should be in the previous commit.
It didn't have to be, because I originally passed NULL in the call to 
`objclass2mo()`. I've just worked in your CR comment to the previous commit 
though, so it'll now be like that.


https://gerrit.osmocom.org/c/osmo-bts/+/32354/comment/5920e8e5_9c37de99
PS2, Line 1106: gsm_objclass2obj
> Ack. […]
That's what I thought. I've changed it now though



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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: If734ea2c8cae4c1f99b02520dffa4e3862a67745
Gerrit-Change-Number: 32354
Gerrit-PatchSet: 3
Gerrit-Owner: arehbein <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-CC: fixeria <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Thu, 20 Apr 2023 23:38:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <[email protected]>
Comment-In-Reply-To: fixeria <[email protected]>
Gerrit-MessageType: comment

Reply via email to