Change in osmo-msc[master]: large refactoring: support inter-BSC and inter-MSC Handover
gerrit-no-reply at lists.osmocom.org
Wed May 8 15:00:16 UTC 2019
Neels Hofmeyr has posted comments on this change. ( https://gerrit.osmocom.org/13137 )
Change subject: large refactoring: support inter-BSC and inter-MSC Handover
Patch Set 9:
PS9, Line 158: rtps = data;
> > it is to clarify the type of the data argument [...] […]
The way I usually do it is define a local variable that indicates the *data type, and then use it in various switch cases. Here it happens to be only the one case...
Why are we arguing about this? :)
PS9, Line 178: struct call_leg *cl = fi->priv;
> Also same opinion, variable is not needed for that: […]
indeed, it's not type safety, it is merely "type safety".
The way I usually do it is assign fi->priv to a local variable of the correct type and then use it in various places of that function. Here it happens to be only the one, and also happens to be passed as a void*.
Nevertheless I prefer to keep that style consistent across the entire file.
PS9, Line 290: call_leg_ensure_rtp_alloc
> Well, call_leg_rtp_alloc() also would just return 0 if it was already called for a given RTP stream. […]
oh. what was I thinking.
PS9, Line 156: DEBUGP
> > This is legacy logging, not sure if we really need to change all of it now? […]
And I would rather share the work load :P
To view, visit https://gerrit.osmocom.org/13137
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-CC: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Comment-Date: Wed, 08 May 2019 15:00:16 +0000
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the gerrit-log