<p><a href="https://gerrit.osmocom.org/13232">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13232/2/src/osmo_ss7_vty.c">File src/osmo_ss7_vty.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13232/2/src/osmo_ss7_vty.c@365">Patch Set #2, Line 365:</a> <code style="font-family:monospace,monospace">                     osmo_ss7_pointcode_print2(rtable->inst, rt->cfg.mask),</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I really don't like having this "2" API with a different static buffer instead of simply copying it  […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I also don't like it and would very much prefer my "volatile select loop iteration talloc context" approach.  However, the existing code works that way and it is out of scope for this patch</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13232">change 13232</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/13232"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-sccp </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ica32e83cbe8af2317cb07f8d8422a399fa537012 </div>
<div style="display:none"> Gerrit-Change-Number: 13232 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 13 Mar 2019 21:20:09 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>