<p> Attention is currently required from: Alexander Traud. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/17587">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File include/asterisk/test.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/+/17587/comment/bd5d0083_b4f02754">Patch Set #1, Line 179:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;"> * \param s The state the application has changed to<br> * \param f The message with format parameters to add to the manager event<br> */<br>#define ast_test_suite_event_notify(s, f, ...) \<br>     __ast_test_suite_event_notify(__FILE__, __PRETTY_FUNCTION__, __LINE__, (s), (f), ## __VA_ARGS__)<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I assume the '...' param is ignored or optional in Doxygen? I noticed in another file you included it (albeit there you were replace current invalid params), but not here.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17587/comment/97c7acae_1e2052ba">Patch Set #1, Line 362:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;"> * \param t currently executing test<br> * \param f printf type format string<br> *<br> * \retval 0 success<br> * \retval -1 failure<br> */<br>#define ast_test_status_update(t, f, ...) __ast_test_status_update(__FILE__, __PRETTY_FUNCTION__, __LINE__, (t), (f), ## __VA_ARGS__)<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">No need for '...' here too?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File main/dns_test.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/+/17587/comment/e94f3611_f239a083">Patch Set #1, Line 148:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">/*!<br> * \brief Write a DNS string to a buffer<br> *<br> * This writes the DNS string to the buffer and returns the total<br> * number of bytes written to the buffer.<br> *<br> * There is no buffer size passed to this function since we provide<br> * the data ourselves and have sized the buffer to be way larger<br> * than necessary for the tests.<br> *<br> * \param string The string to write<br> * \param buf The buffer to write the string into<br> * \return The number of bytes written to the buffer<br> */<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Why is this block removed?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17587/comment/c938cbcb_6c9d1825">Patch Set #1, Line 178:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><br>/*!<br> * \brief Write a DNS domain to a buffer<br> *<br> * A DNS domain consists of a series of labels separated<br> * by dots. Each of these labels gets written as a DNS<br> * string. A DNS domain ends with a NULL label, which is<br> * essentially a zero-length DNS string.<br> *<br> *<br> * There is no buffer size passed to this function since we provide<br> * the data ourselves and have sized the buffer to be way larger<br> * than necessary for the tests.<br> *<br> * \param string The DNS domain to write<br> * \param buf The buffer to write the domain into<br> * \return The number of bytes written to the buffer<br> */<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Same, why remove this one?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17587">change 17587</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/+/17587"/><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: If338163488498f65fa7248b60e80299c0a928e4b </div>
<div style="display:none"> Gerrit-Change-Number: 17587 </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: Friendly Automation </div>
<div style="display:none"> Gerrit-CC: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Attention: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 30 Nov 2021 17:50:59 +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>