[asterisk-commits] kmoore: branch 10 r371437 - in /branches/10: ./ main/loader.c

[SVN commits to the Asterisk project]

Author: kmoore
Date: Fri Aug 17 10:51:06 2012
New Revision: 371437

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=371437
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:
    branches/10/   (props changed)
    branches/10/main/loader.c

Propchange: branches/10/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: branches/10/main/loader.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/main/loader.c?view=diff&rev=371437&r1=371436&r2=371437
==============================================================================
--- branches/10/main/loader.c (original)
+++ branches/10/main/loader.c Fri Aug 17 10:51:06 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 */
 		}
 	}