<p>Joshua Colp <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/7288">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Richard Mudgett: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  Jenkins2: Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">tests: Fix warnings found on Mac.<br><br>test_pbx used raise without explicitly including signal.h.  On Mac for<br>some reason nothing else includes it.<br><br>test_logger checked if an unsigned int was negative.  Switch the<br>variable to 'int' so that error check can be effective.<br><br>Change-Id: Ie1db5dd1818ac25cc2ae41b644f848b5865b1362<br>---<br>M tests/test_logger.c<br>M tests/test_pbx.c<br>2 files changed, 4 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/tests/test_logger.c b/tests/test_logger.c<br>index 59ee3e6..6cbddb9 100644<br>--- a/tests/test_logger.c<br>+++ b/tests/test_logger.c<br>@@ -190,7 +190,7 @@<br> <br> static char *handle_cli_queue_test(struct ast_cli_entry *e, int cmd, struct ast_cli_args *a)<br> {<br>-      unsigned int level;<br>+  int level;<br>    int current_queue_limit;<br>      unsigned int x;<br>       struct timeval start, end;<br>@@ -220,7 +220,7 @@<br>               ast_cli(a->fd, "Test: Failed, could not register level 'queuetest'.\n");<br>                 return CLI_SUCCESS;<br>   }<br>-    ast_cli(a->fd, "Test: got level %u for 'queuetest'.\n", level);<br>+ ast_cli(a->fd, "Test: got level %d for 'queuetest'.\n", level);<br> <br>       if (ast_logger_create_channel(tmppath, "queuetest") != AST_LOGGER_SUCCESS) {<br>                ast_cli(a->fd, "Test: Unable to create logger channel '%s'\n", tmppath);<br>diff --git a/tests/test_pbx.c b/tests/test_pbx.c<br>index 00fa411..155bec9 100644<br>--- a/tests/test_pbx.c<br>+++ b/tests/test_pbx.c<br>@@ -37,6 +37,8 @@<br> #include "asterisk/pbx.h"<br> #include "asterisk/test.h"<br> <br>+#include <signal.h><br>+<br> /*!<br>  * If we determine that we really need<br>  * to be able to register more than 10<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/7288">change 7288</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/7288"/><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: merged </div>
<div style="display:none"> Gerrit-Change-Id: Ie1db5dd1818ac25cc2ae41b644f848b5865b1362 </div>
<div style="display:none"> Gerrit-Change-Number: 7288 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.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-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>