Merged revisions 378823 via svnmerge from
authorAutomerge script <automerge@asterisk.org>
Wed, 9 Jan 2013 22:20:40 +0000 (22:20 +0000)
committerAutomerge script <automerge@asterisk.org>
Wed, 9 Jan 2013 22:20:40 +0000 (22:20 +0000)
file:///srv/subversion/repos/asterisk/trunk

........
  r378823 | rmudgett | 2013-01-09 16:15:41 -0600 (Wed, 09 Jan 2013) | 2 lines

  Tweaked __ast_test_suite_assert_notify() and __ast_test_suite_event_notify() to be void functions.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/team/mmichelson/threadpool@378833 65c4cc65-6c06-0410-ace0-fbb531ad65f3

include/asterisk/test.h
main/test.c

index e8584aa..ea79385 100644 (file)
  * \param state                The state the application has changed to
  * \param fmt          The message with format parameters to add to the manager event
  *
- * \returns 0 on success
- * \returns any other value on failure
+ * \return Nothing
  */
-int __ast_test_suite_event_notify(const char *file, const char *func, int line, const char *state, const char *fmt, ...)
+void __ast_test_suite_event_notify(const char *file, const char *func, int line, const char *state, const char *fmt, ...)
        __attribute__((format(printf, 5, 6)));
 
 /*!
@@ -161,10 +160,9 @@ int __ast_test_suite_event_notify(const char *file, const char *func, int line,
  *
  * \param exp  The expression to evaluate
  *
- * \returns 0 on success
- * \returns any other value on failure
+ * \return Nothing
  */
-int __ast_test_suite_assert_notify(const char *file, const char *func, int line, const char *exp);
+void __ast_test_suite_assert_notify(const char *file, const char *func, int line, const char *exp);
 
 /*!
  * \ref __ast_test_suite_event_notify()
@@ -175,13 +173,17 @@ int __ast_test_suite_assert_notify(const char *file, const char *func, int line,
 /*!
  * \ref __ast_test_suite_assert_notify()
  */
-#define ast_test_suite_assert(exp) \
-       ( (exp) ? (void)0 : __ast_test_suite_assert_notify(__FILE__, __PRETTY_FUNCTION__, __LINE__, #exp))
+#define ast_test_suite_assert(exp)                             \
+       do {                                                                            \
+               if (__builtin_expect(!(exp), 1)) {              \
+                       __ast_test_suite_assert_notify(__FILE__, __PRETTY_FUNCTION__, __LINE__, #exp); \
+               }                                                                               \
+       } while (0)
 
 #else
 
-#define ast_test_suite_event_notify(s, f, ...) (void)0;
-#define ast_test_suite_assert(exp) (void)0;
+#define ast_test_suite_event_notify(s, f, ...)
+#define ast_test_suite_assert(exp)
 
 #endif
 
index 60676cb..612697a 100644 (file)
@@ -910,13 +910,13 @@ static struct ast_cli_entry test_cli[] = {
        AST_CLI_DEFINE(test_cli_generate_results,          "generate test results to file"),
 };
 
-int __ast_test_suite_event_notify(const char *file, const char *func, int line, const char *state, const char *fmt, ...)
+void __ast_test_suite_event_notify(const char *file, const char *func, int line, const char *state, const char *fmt, ...)
 {
        struct ast_str *buf = NULL;
        va_list ap;
 
        if (!(buf = ast_str_create(128))) {
-               return -1;
+               return;
        }
 
        va_start(ap, fmt);
@@ -933,11 +933,9 @@ int __ast_test_suite_event_notify(const char *file, const char *func, int line,
                state, file, func, line, ast_str_buffer(buf));
 
        ast_free(buf);
-
-       return 0;
 }
 
-int __ast_test_suite_assert_notify(const char *file, const char *func, int line, const char *exp)
+void __ast_test_suite_assert_notify(const char *file, const char *func, int line, const char *exp)
 {
        manager_event(EVENT_FLAG_TEST, "TestEvent",
                "Type: Assert\r\n"
@@ -946,8 +944,6 @@ int __ast_test_suite_assert_notify(const char *file, const char *func, int line,
                "AppLine: %d\r\n"
                "Expression: %s\r\n",
                file, func, line, exp);
-
-       return 0;
 }
 
 #endif /* TEST_FRAMEWORK */