Change in osmo-ttcn3-hacks[master]: pcu: Introduce as_ms_rx_ignore_empty helper

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/.

fixeria gerrit-no-reply at lists.osmocom.org
Fri Nov 12 18:19:37 UTC 2021


fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/26220 )

Change subject: pcu: Introduce as_ms_rx_ignore_empty helper
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/26220/1/pcu/GPRS_Components.ttcn 
File pcu/GPRS_Components.ttcn:

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/26220/1/pcu/GPRS_Components.ttcn@562 
PS1, Line 562: inout GprsMS ms
> It's the same as with as_ms_rx_ignore_dummy above. […]
I expected this kind of answer, but neither dummy nor empty DATA.req messages contain MS specific info. I simply don't see how GprsMS might be used in the context of empty DATA.req messages, or how can this altstep might evolve in the future so that GprsMS is needed. And same applies to as_ms_rx_ignore_dummy() - requiring the API user to pass aruments for no real need is a bad idea. IMO, both altsteps should not require GprsMS and rather use "as_pcuif_" prefix.



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

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I3c80ba4b8d229fa4a75dfd04abb7c5aae26e221c
Gerrit-Change-Number: 26220
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Comment-Date: Fri, 12 Nov 2021 18:19:37 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin at sysmocom.de>
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/20211112/2d7ff3b2/attachment.htm>


More information about the gerrit-log mailing list