----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3576/#review12072 -----------------------------------------------------------
branches/12/main/astobj2.c <https://reviewboard.asterisk.org/r/3576/#comment22063> Need to restore paths.h include for REF_DEBUG. #include "asterisk/paths.h" branches/12/main/astobj2.c <https://reviewboard.asterisk.org/r/3576/#comment22064> stray blank here branches/12/main/astobj2_container.c <https://reviewboard.asterisk.org/r/3576/#comment22065> stray blank here branches/12/main/astobj2_container.c <https://reviewboard.asterisk.org/r/3576/#comment22066> stray blank here branches/12/main/astobj2_container.c <https://reviewboard.asterisk.org/r/3576/#comment22067> Swap these two lines. i.e. put blank after the endif. - rmudgett On June 5, 2014, 6:43 p.m., George Joseph wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3576/ > ----------------------------------------------------------- > > (Updated June 5, 2014, 6:43 p.m.) > > > Review request for Asterisk Developers and rmudgett. > > > Repository: Asterisk > > > Description > ------- > > In preparation for weak-reference containers, and because it makes the > existing code easier to read and maintain, I've split the astobj2 common > structure and enum definitions and prototypes into astobj2_private.h, the > hash table implementation into astobj2_hash.c, and the rbtree implementation > into astobj2_rbtree.c. All of the public functions remain in astobj2.c. > > A few functions (adjust_lock, container_destruct, container_destruct_debug) > needed to have their static modifiers removed so they'd be visible from the > other object files but other than that there were NO functional changes, no > logic changes, etc. > > EDIT:.. > Also added a basic test to the test framework to monitor performance impacts > as changes are made to astobj2. > > > Diffs > ----- > > branches/12/utils/Makefile 415300 > branches/12/tests/test_astobj2.c 415300 > branches/12/main/astobj2_rbtree.c PRE-CREATION > branches/12/main/astobj2_private.h PRE-CREATION > branches/12/main/astobj2_hash.c PRE-CREATION > branches/12/main/astobj2_container_private.h PRE-CREATION > branches/12/main/astobj2_container.c PRE-CREATION > branches/12/main/astobj2.c 415300 > branches/12/include/asterisk/astobj2.h 415300 > > Diff: https://reviewboard.asterisk.org/r/3576/diff/ > > > Testing > ------- > > I used both the test framework and the test suite. For the test suite, I > used channels/pjsip since that exercises sorcery significantly and that in > turn exercises astobj2. > > All tests that worked before the change worked after the change. > > Before... > > Test Framework > 393 Test(s) Executed 393 Passed 0 Failed > > Test Suite > tests/channels/pjsip/ > Tests: 88 Passed: 87 Failed: 1 > FAILED: tests/channels/pjsip/dialplan_functions/pjsip_endpoint > > After... > > Test Framework > 393 Test(s) Executed 393 Passed 0 Failed > > Test Suite > tests/channels/pjsip/ > Tests: 88 Passed: 87 Failed: 1 > FAILED: tests/channels/pjsip/dialplan_functions/pjsip_endpoint > > Not sure why the pjsip_endpoint function is failing but it's not this patch's > fault. > > > Thanks, > > George Joseph > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
