Review Board 1.7.16


Add a _remove method for routes on a message_router.

Review Request #2591 - Created June 4, 2013 and submitted

Jason Parker
/trunk/
Reviewers
asterisk-dev
dlee
Asterisk
Requested on review 2586.

Convert router routes to ao2_containers and add a way of removing them.
Stuff didn't crash when I unloaded my module that removes routes.  Messages still got routed through AMI.

Changes between revision 2 and 3

1 2 3
1 2 3

  1. /trunk/include/asterisk/stasis_message_router.h: Loading...
  2. /trunk/main/stasis_message_router.c: Loading...
/trunk/include/asterisk/stasis_message_router.h
Diff Revision 2 Diff Revision 3
[20] 113 lines
[+20] [+] int stasis_message_router_add(struct stasis_message_router *router,
114
 * \brief Remove a route from a message router.
114
 * \brief Remove a route from a message router.
115
 *
115
 *
116
 * \param router Router to remove the route from.
116
 * \param router Router to remove the route from.
117
 * \param message_type Type of message to route.
117
 * \param message_type Type of message to route.
118
 *
118
 *
119
 * \retval 0 on success

   
120
 * \retval -1 on failure

   
121
 *

   
122
 * \since 12
119
 * \since 12
123
 */
120
 */
124
int stasis_message_router_remove(struct stasis_message_router *router,
121
void stasis_message_router_remove(struct stasis_message_router *router,
125
			      struct stasis_message_type *message_type);
122
			      struct stasis_message_type *message_type);
126

    
   
123

   
127
/*!
124
/*!
128
 * \brief Sets the default route of a router.
125
 * \brief Sets the default route of a router.
129
 *
126
 *
[+20] [20] 14 lines
/trunk/main/stasis_message_router.c
Diff Revision 2 Diff Revision 3
 
  1. /trunk/include/asterisk/stasis_message_router.h: Loading...
  2. /trunk/main/stasis_message_router.c: Loading...

https://reviewboard.asterisk.org/ runs on a server provided by Digium, Inc. and uses bandwidth donated to the open source Asterisk community by API Digital Communications in Huntsville, AL USA.
Please report problems with this site to asteriskteam@digium.com.