<p>Corey Farrell has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/7290">View Change</a></p><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;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/90/7290/1</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 bf809ba..08585a6 100644<br>--- a/tests/test_logger.c<br>+++ b/tests/test_logger.c<br>@@ -192,7 +192,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>@@ -222,7 +222,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 acf7484..df6b88d 100644<br>--- a/tests/test_pbx.c<br>+++ b/tests/test_pbx.c<br>@@ -39,6 +39,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/7290">change 7290</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/7290"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Ie1db5dd1818ac25cc2ae41b644f848b5865b1362 </div>
<div style="display:none"> Gerrit-Change-Number: 7290 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </div>