<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1: Code-Review-1</p><p style="white-space: pre-wrap; word-wrap: break-word;">I am sorry, but what's the point of this change? I believe the configuration examples should contain all available options, so there is no need to refer documentation / source code every time you need to remember position / format of some option. Look at configuration examples of srsLTE - everything is listed and documented there, so you don't have to leave your text editor at all.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">But this is exactly not how we deal with configuration in osmocom.</p><ul><li>we do have compile-time default</li><li>even a 'write file' doesn't save all values, only those that differ from default</li><li>we do not rely on text editing of config files but have the VTY to interactively change it</li></ul><p style="white-space: pre-wrap; word-wrap: break-word;">Yes, this has it's problems and it can be debated.  But then please start a related issue or mailing list thread for the discussion, and at the end of that we change all of the codebase.  This specific issue is bringing the config file example in-line with the existing practice.</p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +2</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-trx/+/17595">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-trx/+/17595">change 17595</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-trx/+/17595"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-trx </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I105d1c51424836daa6893e83a81c83cc7ac6afd4 </div>
<div style="display:none"> Gerrit-Change-Number: 17595 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 27 Mar 2020 09:26:34 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>