[test-results] [Bamboo] Asterisk - 1.8 > Mac OS X Snow Leopard (10.6) > #332 was SUCCESSFUL (with 64 tests). Change made by Terry Wilson, rmudgett and kmoore.

Bamboo bamboo at asterisk.org
Wed Oct 26 03:35:36 CDT 2011


-----------------------------------------------------------------------
Asterisk - 1.8 > Mac OS X Snow Leopard (10.6) > #332 was successful.
-----------------------------------------------------------------------
This build occurred because it is a dependant of AST18-LUCID-885.
64 tests in total.

http://bamboo.asterisk.org/browse/AST18-SNOWLEOPARD-332/


--------------
Code Changes
--------------
Terry Wilson (342383):

>Simplify queue membercount code
>
>Despite an ominous sounding comment stating that membercount was for "logged
>in" members only and thus we couldn't use ao2_container_count(), I could not
>find a single place in the code where that seemed to be accurate. The only time
>we decremented membercount was when we were marking something dead or actually
>removing it. The only places we incremented it were either after ao2_link(), or
>trying to correct for having set it to 0 during a reload. In every case where
>we were correcting the value, it seemed that we were trying to make the count
>actually match what ao2_container_count() would return. The only place I could
>find where we made a determination about something being "logged in" or not, we
>didn't trust the membercount, but instead looked at devicestate, paused, etc.
>
>This patch removes membercount, replaces its use with ao2_container_count, and
>manually adds the results of ao2_container_count to a "membercount" field for
>ast_data queue query results. This patch also would fix AST-676, but as it is
>slightly riskier than the previously committed fix, the two commits have been
>made separately.
>
>Reivew: https://reviewboard.asterisk.org/r/1541/
>

Terry Wilson (342435):

>Use int for storing ao2_container_count instad of size_t
>
>AST-676
>

rmudgett (342487):

>Check fopen return value for ao2 reference debug output.
>
>Reported by: wdoekes
>Patched by: wdoekes
>
>Review: https://reviewboard.asterisk.org/r/1539/
>


--
This message is automatically generated by Atlassian Bamboo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/test-results/attachments/20111026/5e44da0a/attachment.htm>


More information about the Test-results mailing list