<p> Attention is currently required from: Mark Murawski. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/17655">View Change</a></p><p>7 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</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.asterisk.org/c/asterisk/+/17655?tab=comments">Patch Set #4:</a> </p><p style="white-space: pre-wrap; word-wrap: break-word;">This patch triggers whitespace warnings when applied - I've noted inline where these issues exist:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Gerrit Patchset: https://gerrit.asterisk.org/changes/asterisk~17655/revisions/4/patch?download<br>Downloading and applying Gerrit patch 17655<br>17655.diff:1130: trailing whitespace.<br>        struct group_data_store *group_store;          /*!< The group_data_store we live on */<br>17655.diff:1660: trailing whitespace.<br>        struct group_data_store *group_store;          /*!< The group_data_store we live on. (Type: struct tie_data_store *) */<br>warning: 2 lines add whitespace errors.</pre><p style="white-space: pre-wrap; word-wrap: break-word;">A few additional comments below, in addition to those still pending:</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File funcs/func_groupcount.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.asterisk.org/c/asterisk/+/17655/comment/0aa4e7c5_c18b3e6d">Patch Set #4, Line 622:</a> <code style="font-family:monospace,monospace">//        const char *group    = astman_get_header(m, "Group");  // todo: filtering</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">If these variables aren't needed, get rid of them</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17655/comment/4b9f9d55_369607c2">Patch Set #4, Line 660:</a> <code style="font-family:monospace,monospace">//   const char *group = astman_get_header(m, "Group");     // todo: filtering</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Same thing here</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File main/app.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.asterisk.org/c/asterisk/+/17655/comment/724ecacd_83952e9f">Patch Set #4, Line 131:</a> <code style="font-family:monospace,monospace">    struct group_data_store *group_store;          /*!< The group_data_store we live on */ </code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Trailing whitespace</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17655/comment/b2c61252_72da7216">Patch Set #4, Line 2593:</a> <code style="font-family:monospace,monospace">                manager_event(EVENT_FLAG_DIALPLAN, "GroupDestroy",</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't see events like this documented anywhere.<br>Every AMI event should have documentation.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17655/comment/40f83a8a_3c042c49">Patch Set #4, Line 2703:</a> <code style="font-family:monospace,monospace">                strncpy(gi->category, new_category, MAX_CATEGORY_LEN);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">You need to address this compiler warning:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">app.c:2707:17: warning: ‘strncpy’ specified bound 80 equals destination size [-Wstringop-truncation]<br>                 strncpy(gmi->category, new_category, MAX_CATEGORY_LEN);</pre></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File main/cli.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.asterisk.org/c/asterisk/+/17655/comment/fd765871_761762ba">Patch Set #4, Line 1906:</a> <code style="font-family:monospace,monospace"> struct group_data_store *group_store;          /*!< The group_data_store we live on. (Type: struct tie_data_store *) */ </code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Trailing whitespace<br>Also, get rid of the (Type: struct tie_data_store)</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17655">change 17655</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.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.asterisk.org/c/asterisk/+/17655"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 18 </div>
<div style="display:none"> Gerrit-Change-Id: I23e48d1cdfc8adaffdfec2e936e56143603914f2 </div>
<div style="display:none"> Gerrit-Change-Number: 17655 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Mark Murawski <markm@intellasoft.net> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-Attention: Mark Murawski <markm@intellasoft.net> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 19 Feb 2022 13:40:24 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>