[asterisk-commits] mmichelson: trunk r292742 - in /trunk: ./ tests/test_event.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Fri Oct 22 12:11:02 CDT 2010


Author: mmichelson
Date: Fri Oct 22 12:10:59 2010
New Revision: 292742

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=292742
Log:
Merged revisions 292741 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r292741 | mmichelson | 2010-10-22 12:09:52 -0500 (Fri, 22 Oct 2010) | 12 lines
  
  Prevent multiple runs of event_sub_test from producing false failure results.
  
  The array of test subscriptions was declared "static," meaning that the
  data.count field would retain its value between runs of the test. After the
  first test run, this would result in false reports of test failures.
  
  I chose to just remove the "static" keyword from the structure since it's not
  a huge deal to construct this structure during each run of the test. Another
  alternative would have been to zero out the data.count fields of each test
  subscription instead.
........

Modified:
    trunk/   (props changed)
    trunk/tests/test_event.c

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

Modified: trunk/tests/test_event.c
URL: http://svnview.digium.com/svn/asterisk/trunk/tests/test_event.c?view=diff&rev=292742&r1=292741&r2=292742
==============================================================================
--- trunk/tests/test_event.c (original)
+++ trunk/tests/test_event.c Fri Oct 22 12:10:59 2010
@@ -253,7 +253,7 @@
 	struct ast_event *event;
 	int i;
 	enum ast_event_subscriber_res sub_res;
-	static struct {
+	struct {
 		struct ast_event_sub *sub;
 		struct event_sub_data data;
 		const unsigned int expected_count;




More information about the asterisk-commits mailing list