No subject


Thu Jul 12 09:23:04 CDT 2007


one in handle_statechange() and the other one in update_status(). I've
removed the first one and I came up with a simpler form of
handle_statechange. This just passes the state interface name to the
update_status. I've also removed "/" checking in update_status(), so now
Custom: state interfaces are working 100%.

The real application I'm using is emulation of chan_agent with MeetMe
rooms, as AgentLogin does not have dynamic capabilities and I need "live"
agents (always on).


====================================================================== 

---------------------------------------------------------------------- 
 (0092264) svnbot (reporter) - 2008-09-09 12:06
 http://bugs.digium.com/view.php?id=12979#c92264 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 142146

U   trunk/apps/app_queue.c

------------------------------------------------------------------------
r142146 | mmichelson | 2008-09-09 12:05:59 -0500 (Tue, 09 Sep 2008) | 21
lines

This is the trunk version of the patch to close
issue 12979. The difference between this and the
1.6.0 and 1.6.1 versions is that this is a much more
invasive change. With this, we completely get rid
of the interfaces list, along with all its helper
functions.

Let me take a moment to say that this change personally
excites me since it may mean huge steps forward regarding
proper lock order in app_queue without having to strew
seemingly unnecessary locks all over the place. It also
results in a huge reduction in lines of code and complexity.
Way to go Brett!

(closes issue http://bugs.digium.com/view.php?id=12979)
Reported by: sigxcpu
Patches:
      20080710_issue12979_queue_custom_state_interface_trunk_2.diff
uploaded by bbryant (license 36)
Tested by: sigxcpu, putnopvut


------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=142146 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-09-09 12:06 svnbot         Note Added: 0092264                          
======================================================================




More information about the asterisk-bugs mailing list