<p>Alexander Traud <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/9195">View Change</a></p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p style="white-space: pre-wrap; word-wrap: break-word;">The typedef itself is not redundant because the local header uses it but does not know about it, yet. The build system makes it redundant. If you want to avoid the typedef, we have to look into how to solve that.</p><p style="white-space: pre-wrap; word-wrap: break-word;">It would be cool if the Asterisk/Digium folks could start using Git, Jira, and Gerrit. This issue was introduced by me recently. In Git, you look up which change to blame (works even in the GitHub mirror). Then, you create an issue report on Jira and link that to the original one. Then in Gerrit, you invite the one who introduced that change. That way, the original author can is notified and can double-check. If the fix is correct, the original author is able to learn and improve. If the fix is incorrect/incomplete, the original author is able to block it.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/9195">change 9195</a>. To unsubscribe, 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/9195"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 15 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I5a7d4565990c985822d61ce75bde0b45f9870540 </div>
<div style="display:none"> Gerrit-Change-Number: 9195 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 20 Jun 2018 11:04:50 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>