Max has posted comments on this change. ( https://gerrit.osmocom.org/12519 )
Change subject: Clarify transaction ID assignment ...................................................................... Patch Set 4: > since we I've decided to get rid of this flag When? I don't recall such decision. I've proposed to remove it since it was neither used nor documented so it only added confusion. However, as it's part of the spec with clearly defined meaning than I don't mind keeping it if it's properly documented. I also don't mind removing it. The former already have a patch ready to be merged. And it doesn't prevent us from removing it if we'll actually settle on this decision. -- To view, visit https://gerrit.osmocom.org/12519 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie11999900b1789652ee078d34636dcda1e137eb0 Gerrit-Change-Number: 12519 Gerrit-PatchSet: 4 Gerrit-Owner: Max <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-Reviewer: Max <[email protected]> Gerrit-Reviewer: Neels Hofmeyr <[email protected]> Gerrit-Reviewer: Vadim Yanitskiy <[email protected]> Gerrit-Reviewer: osmith <[email protected]> Gerrit-Comment-Date: Tue, 29 Jan 2019 13:50:54 +0000 Gerrit-HasComments: No Gerrit-HasLabels: No
