<p> Attention is currently required from: N A. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/18885">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File channels/iax2/include/iax2.h:</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/+/18885/comment/9a45c545_365f1ddd">Patch Set #2, Line 131:</a> <code style="font-family:monospace,monospace">#define IAX_IE_CALLED_NUMBER                   1               /*!< Number/extension being called - string */</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Okay, I see what you're seeing now, I think, which is definitely kind off. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Tabs after the first non-whitespace will ALWAYS be an issue because of the disagreement of how to expand tabs in different environments.  But this is the Asterisk coding format.  It is better to have comments above the item being described rather than having end-of-line comments that need to line up to look good.  Putting the comments above avoids most of the issues with tabs in code presentation.  A bonus of commenting above the item is you have more room for the comment before hitting the 90 column limit.</p><p style="white-space: pre-wrap; word-wrap: break-word;">/*! Calling number - string */<br>#define IAX_IE_CALLING_NUMBER    2</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">struct foo {<br>    /*! This member does blah */<br>    int member;<br>    /*! This member does something else that is needed */<br>    int bar;<br>};</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">/*!<br> * \brief Do this foo thing<br> *<br> * \param bar How much to do something with foo.<br> *<br> * \retvar -1 on error is not always true.  People have a tendency to assume this.<br> */<br>int foo(int bar)<br>{<br>    /*! To do foo's things */<br>    int another_foo;<br>    /*! Index into some foo array */<br>    int idx;</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">    return -1;<br>}</pre></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/18885">change 18885</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/+/18885"/><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-Change-Id: I07771c4c89facd41ce8d323859f022ddbddf6ca7 </div>
<div style="display:none"> Gerrit-Change-Number: 18885 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 03 Aug 2022 12:59:22 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Comment-In-Reply-To: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>