Patch Set 2: (1 comment)
https://gerrit.osmocom.org/#/c/2143/2/Transceiver52M/osmo-trx.cpp File Transceiver52M/osmo-trx.cpp: Line 419: case 'E': > So user can still specify ... -E 123 -r 321 ... without enabling EDGE and i In general, I would very much like to stop adding more and more command line arguments. There are already other ambiguous cases similar to that mentioned by Max and most users never use them. I'm open to ideas (e.g. config file, string-value arguments?). -- To view, visit https://gerrit.osmocom.org/2143 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic8808bbe3f06740ef3fec1d1865ecb57fbcfabab Gerrit-PatchSet: 2 Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Owner: Alexander Chemeris <alexander.cheme...@gmail.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Max <msur...@sysmocom.de> Gerrit-Reviewer: Tom Tsou <t...@tsou.cc> Gerrit-HasComments: Yes