<p>Piotr Krysik <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/10420">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Piotr Krysik: Looks good to me, approved; Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">apps/grgsm_trx: use format string for help message<br><br>Instead of using the hard-coded default values in help message,<br>it makes sense to use a format string, and pass the actual<br>values when printing help.<br><br>Change-Id: Ib1bf0ef3ded86aa92faeb9b63eb286283f5c8c3d<br>---<br>M apps/grgsm_trx<br>1 file changed, 15 insertions(+), 8 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/apps/grgsm_trx b/apps/grgsm_trx</span><br><span>index eb7fbdb..b925fc1 100755</span><br><span>--- a/apps/grgsm_trx</span><br><span>+++ b/apps/grgsm_trx</span><br><span>@@ -96,20 +96,27 @@</span><br><span> </span><br><span>               # TRX specific</span><br><span>               s += " TRX interface specific\n" \</span><br><span style="color: hsl(0, 100%, 40%);">-                     "  -i --remote-addr  Set remote address (default 127.0.0.1)\n" \</span><br><span style="color: hsl(0, 100%, 40%);">-                      "  -p --base-port    Set base port number (default 5700)\n\n"</span><br><span style="color: hsl(120, 100%, 40%);">+                       "  -i --remote-addr  Set remote address (default %s)\n" \</span><br><span style="color: hsl(120, 100%, 40%);">+                   "  -p --base-port    Set base port number (default %d)\n\n"</span><br><span> </span><br><span>           # PHY specific</span><br><span>               s += " Radio interface specific\n" \</span><br><span>                        "  -a --device-args  Set device arguments\n" \</span><br><span>                     "  -s --sample-rate  Set sample rate\n" \</span><br><span style="color: hsl(0, 100%, 40%);">-                     "  -g --rx-gain      Set RX gain (default 30)\n" \</span><br><span style="color: hsl(0, 100%, 40%);">-                    "  -G --tx-gain      Set TX gain (default 10)\n" \</span><br><span style="color: hsl(0, 100%, 40%);">-                    "     --rx-antenna   Set RX antenna (default RX2)\n" \</span><br><span style="color: hsl(0, 100%, 40%);">-                        "     --tx-antenna   Set TX antenna (default TX/RX)\n" \</span><br><span style="color: hsl(0, 100%, 40%);">-                      "     --ppm          Set frequency correction (default 0)\n"</span><br><span style="color: hsl(120, 100%, 40%);">+                        "  -g --rx-gain      Set RX gain (default %d)\n" \</span><br><span style="color: hsl(120, 100%, 40%);">+                  "  -G --tx-gain      Set TX gain (default %d)\n" \</span><br><span style="color: hsl(120, 100%, 40%);">+                  "     --rx-antenna   Set RX antenna (default %s)\n" \</span><br><span style="color: hsl(120, 100%, 40%);">+                       "     --tx-antenna   Set TX antenna (default %s)\n" \</span><br><span style="color: hsl(120, 100%, 40%);">+                       "     --ppm          Set frequency correction (default %d)\n"</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-            print(s)</span><br><span style="color: hsl(120, 100%, 40%);">+              print(s % (</span><br><span style="color: hsl(120, 100%, 40%);">+                   self.remote_addr,</span><br><span style="color: hsl(120, 100%, 40%);">+                     self.base_port,</span><br><span style="color: hsl(120, 100%, 40%);">+                       self.phy_rx_gain,</span><br><span style="color: hsl(120, 100%, 40%);">+                     self.phy_tx_gain,</span><br><span style="color: hsl(120, 100%, 40%);">+                     self.phy_rx_antenna,</span><br><span style="color: hsl(120, 100%, 40%);">+                  self.phy_tx_antenna,</span><br><span style="color: hsl(120, 100%, 40%);">+                  self.phy_ppm))</span><br><span> </span><br><span>   def parse_argv(self):</span><br><span>                try:</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/10420">change 10420</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/10420"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: gr-gsm </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: Ib1bf0ef3ded86aa92faeb9b63eb286283f5c8c3d </div>
<div style="display:none"> Gerrit-Change-Number: 10420 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Piotr Krysik <ptrkrysik@gmail.com> </div>