osmo-iuh[master]: cs RAB: add nAS_SynchronisationIndicator

Holger Freyther gerrit-no-reply at lists.osmocom.org
Wed Feb 1 06:59:00 UTC 2017


Patch Set 1:

> I have no idea really; I saw it in a real-world 3G voice trace, and
 > there was a "fixme" comment in the code to add it... I added it in
 > an attempt to fix the nano3G, but it had no effect. Now it's there
 > so we might as well keep it to be more like the real-world trace.

that is good enough. Please put it into the commit message of each
commit or create a merge commit that explains it in the merge once.

The reason is.. in a year or two we will look at it.. know the specific
meaning of these fields more and and wonder why we did this...

-- 
To view, visit https://gerrit.osmocom.org/1642
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I14c22b0befb308bac2eded662fe13a58c8478743
Gerrit-PatchSet: 1
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-HasComments: No


More information about the gerrit-log mailing list