<p>Corey Farrell <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;"><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. 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 extensions.  This applies to modules as well.  The GCC documentation you linked on the issue mentions "general-purpose header file".  None of the Asterisk headers are general purpose, they are for Asterisk only.  Since we require GCC extensions I feel 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.  I'm suspecting net-snmp in FreeBSD is telling us to add -std=c99 or similar to CFLAGS.  This is why I asked for your makeopts and noisy 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 readability.  In master we use typeof in 94 lines of header files, none of them currently use __typeof__.  In addition I feel that __typeof__ is less readable so I'm not thrilled about the idea of using it.</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-Comment-Date: Sun, 28 Jan 2018 19:30:22 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>