[asterisk-commits] kmoore: branch certified-1.8.15 r371440 - in /certified/branches/1.8.15: ./ m...

[SVN commits to the Asterisk project]

Author: kmoore
Date: Fri Aug 17 11:02:50 2012
New Revision: 371440

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=371440
Log:
Add instrumentation to subsystem reloads

When Asterisk is built with TEST_FRAMEWORK defined, Asterisk will now
generate TestEvent AMI events on subsystem reloads such as cdr, dnsmgr,
extconfig, etc.

(issue PQ-1126)
........

Merged revisions 371436 from http://svn.asterisk.org/svn/asterisk/branches/1.8

Modified:
    certified/branches/1.8.15/   (props changed)
    certified/branches/1.8.15/main/loader.c

Propchange: certified/branches/1.8.15/
------------------------------------------------------------------------------
--- branch-1.8-merged (original)
+++ branch-1.8-merged Fri Aug 17 11:02:50 2012
@@ -1,1 +1,1 @@
-/branches/1.8:1-369921,370769,371141,371201,371393
+/branches/1.8:1-369921,370769,371141,371201,371393,371436

Modified: certified/branches/1.8.15/main/loader.c
URL: http://svnview.digium.com/svn/asterisk/certified/branches/1.8.15/main/loader.c?view=diff&rev=371440&r1=371439&r2=371440
==============================================================================
--- certified/branches/1.8.15/main/loader.c (original)
+++ certified/branches/1.8.15/main/loader.c Fri Aug 17 11:02:50 2012
@@ -713,7 +713,9 @@
 	/* Call "predefined" reload here first */
 	for (i = 0; reload_classes[i].name; i++) {
 		if (!name || !strcasecmp(name, reload_classes[i].name)) {
-			reload_classes[i].reload_fn();	/* XXX should check error ? */
+			if (!reload_classes[i].reload_fn()) {
+				ast_test_suite_event_notify("MODULE_RELOAD", "Message: %s", name);
+			}
 			res = 2;	/* found and reloaded */
 		}
 	}