<p>Matthew Fredrickson <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8061">View Change</a></p><p>Patch set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">The usage of typeof is not recommended by GCC within headers. We<br>should avoid that for sure. Asterisk should be a good role model.<br>I<br>see no reason why that should block this review.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">The Asterisk README states that we require a compiler with GCC<br>extensions.  This applies to modules as well.  The GCC<br>documentation you linked on the issue mentions "general-purpose<br>header file".  None of the Asterisk headers are general purpose,<br>they are for Asterisk only.  Since we require GCC extensions I feel<br>it is not a bug for Asterisk to use GCC extensions in the headers.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Something is disabling GCC extensions and that is the actual bug. <br>I'm suspecting net-snmp in FreeBSD is telling us to add -std=c99 or<br>similar to CFLAGS.  This is why I asked for your makeopts and noisy<br>build so we can find out why res_snmp is misbehaving.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Two specific reasons I don't want this change: consistency and<br>readability.  In master we use typeof in 94 lines of header files,<br>none of them currently use __typeof__.  In addition I feel that<br>__typeof__ is less readable so I'm not thrilled about the idea of<br>using it.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Bump.  Alexander, were you planning on responding to Corey's comments?</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8061">change 8061</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/8061"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I0d7a2c62ed256cca19a50340e3c337567ac78e58 </div>
<div style="display:none"> Gerrit-Change-Number: 8061 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Matthew Fredrickson <creslin@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 16 Feb 2018 15:37:52 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>