Minor parking cleanup.
authorRichard Mudgett <rmudgett@digium.com>
Thu, 15 Aug 2013 21:52:01 +0000 (21:52 +0000)
committerRichard Mudgett <rmudgett@digium.com>
Thu, 15 Aug 2013 21:52:01 +0000 (21:52 +0000)
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@396812 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/features.c
res/parking/parking_tests.c
res/parking/res_parking.h
res/res_parking.c

index d4ecf65..70c5423 100644 (file)
@@ -232,8 +232,6 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
        </manager>
  ***/
 
-/* TODO Scrape all of the parking stuff out of features.c */
-
 typedef enum {
        FEATURE_INTERPRET_DETECT, /* Used by ast_feature_detect */
        FEATURE_INTERPRET_DO,     /* Used by feature_interpret */
index 1b1cfc1..57edd37 100644 (file)
@@ -804,42 +804,32 @@ cleanup:
 #endif /* TEST_FRAMEWORK */
 
 
-int unload_parking_tests(void)
+void unload_parking_tests(void)
 {
 /* NOOP without test framework */
 #if defined(TEST_FRAMEWORK)
-
-       int res = 0;
-
-       res |= AST_TEST_UNREGISTER(create_lot);
-       res |= AST_TEST_UNREGISTER(park_call);
-       res |= AST_TEST_UNREGISTER(retrieve_call);
-       res |= AST_TEST_UNREGISTER(park_extensions);
-       res |= AST_TEST_UNREGISTER(extension_conflicts);
-       res |= AST_TEST_UNREGISTER(dynamic_parking_variables);
-
-       return res;
+       AST_TEST_UNREGISTER(create_lot);
+       AST_TEST_UNREGISTER(park_call);
+       AST_TEST_UNREGISTER(retrieve_call);
+       AST_TEST_UNREGISTER(park_extensions);
+       AST_TEST_UNREGISTER(extension_conflicts);
+       AST_TEST_UNREGISTER(dynamic_parking_variables);
 #endif
-
-       return 0;
 }
 
 int load_parking_tests(void)
 {
-/* NOOP without test framework */
-#if defined(TEST_FRAMEWORK)
-
        int res = 0;
 
+/* NOOP without test framework */
+#if defined(TEST_FRAMEWORK)
        res |= AST_TEST_REGISTER(create_lot);
        res |= AST_TEST_REGISTER(park_call);
        res |= AST_TEST_REGISTER(retrieve_call);
        res |= AST_TEST_REGISTER(park_extensions);
        res |= AST_TEST_REGISTER(extension_conflicts);
        res |= AST_TEST_REGISTER(dynamic_parking_variables);
-
-       return res;
 #endif
 
-       return 0;
+       return res;
 }
index b376a99..5dc99b7 100644 (file)
@@ -568,7 +568,6 @@ int load_parking_tests(void);
  * \since 12.0.0
  * \brief Unregister parking unit tests
  *
- * \retval 0 on success
- * \retval nonzero on failure
+ * \return Nothing
  */
-int unload_parking_tests(void);
+void unload_parking_tests(void);
index 668ed63..c7156fb 100644 (file)
@@ -196,8 +196,6 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 #define PARKED_CALL_APPLICATION "ParkedCall"
 #define PARK_AND_ANNOUNCE_APPLICATION "ParkAndAnnounce"
 
-/* TODO Add unit tests for parking */
-
 static int parking_lot_sort_fn(const void *obj_left, const void *obj_right, int flags)
 {
        const struct parking_lot *left = obj_left;