Change in osmo-pcu[master]: pdch::rcv_resource_request(): Use local var to store bts pointer

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

pespin gerrit-no-reply at lists.osmocom.org
Fri Nov 5 12:54:54 UTC 2021


pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/26078 )

Change subject: pdch::rcv_resource_request(): Use local var to store bts pointer
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.osmocom.org/c/osmo-pcu/+/26078/1//COMMIT_MSG 
Commit Message:

https://gerrit.osmocom.org/c/osmo-pcu/+/26078/1//COMMIT_MSG@7 
PS1, Line 7: pdch::rcv_resource_request(): Use local var to store bts pointer
> So what's your motivation, and how it makes the code better?
Because I'm adding even more uses of bts() since I'm adding new counters, and there's no point in calling bts() each time. Makes code clearer, and is also a preparation to make transition to C code easier later.



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

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Ica727c4b70cecc0ddb5d2a235bfc416735754b61
Gerrit-Change-Number: 26078
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Comment-Date: Fri, 05 Nov 2021 12:54:54 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy at sysmocom.de>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211105/476cb532/attachment.htm>


More information about the gerrit-log mailing list