[asterisk-commits] mmichelson: branch group/CCSS r246668 - in /team/group/CCSS: channels/ includ...

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Mon Feb 15 16:08:10 CST 2010


Author: mmichelson
Date: Mon Feb 15 16:08:06 2010
New Revision: 246668

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=246668
Log:
First commit dealing with name consistency issues.

Use the term "dialstring" instead of "dialable_name"
from now on.


Modified:
    team/group/CCSS/channels/chan_local.c
    team/group/CCSS/include/asterisk/ccss.h
    team/group/CCSS/main/ccss.c

Modified: team/group/CCSS/channels/chan_local.c
URL: http://svnview.digium.com/svn/asterisk/team/group/CCSS/channels/chan_local.c?view=diff&rev=246668&r1=246667&r2=246668
==============================================================================
--- team/group/CCSS/channels/chan_local.c (original)
+++ team/group/CCSS/channels/chan_local.c Mon Feb 15 16:08:06 2010
@@ -690,7 +690,7 @@
 	}
 	snprintf(full_dest, sizeof(full_dest), "Local/%s", dest);
 	ast_set_cc_interfaces_chanvar(p->chan, reduced_dest);
-	ast_cc_set_extension_dialable_name(p->chan, full_dest);
+	ast_cc_set_extension_dialstring(p->chan, full_dest);
 
 	/* Start switch on sub channel */
 	if (!(res = ast_pbx_start(p->chan)))

Modified: team/group/CCSS/include/asterisk/ccss.h
URL: http://svnview.digium.com/svn/asterisk/team/group/CCSS/include/asterisk/ccss.h?view=diff&rev=246668&r1=246667&r2=246668
==============================================================================
--- team/group/CCSS/include/asterisk/ccss.h (original)
+++ team/group/CCSS/include/asterisk/ccss.h Mon Feb 15 16:08:06 2010
@@ -101,7 +101,7 @@
  *
  * \param chan The channel onto which to queue the frame
  * \param monitor_type The type of monitor to use when CC is requested
- * \param dialable_name The dial string used to call the device
+ * \param dialstring The dial string used to call the device
  * \param service The type of CC service the device is willing to offer
  * \param private_data If a native monitor is being used, and some channel-driver-specific private
  * data has been allocated, then this parameter should contain a pointer to that data. If using a generic
@@ -111,7 +111,7 @@
  * \retval -1 Error
  */
 int ast_queue_cc_frame(struct ast_channel *chan, const char * const monitor_type,
-		const char * const dialable_name, enum ast_cc_service_type service, void *private_data);
+		const char * const dialstring, enum ast_cc_service_type service, void *private_data);
 
 /*!
  * \brief Allocate and initialize an ast_cc_config_params structure
@@ -506,7 +506,7 @@
 	 * to recognize as being a CC recall. Similarly, ISDN will need a specific
 	 * dial string to know that the call is a recall.
 	 */
-	char *dialable_name;
+	char *dialstring;
 	/*!
 	 * The ID of the available timer used by the current monitor
 	 */
@@ -662,7 +662,7 @@
  * will never be used outside of that file, so you can safely ignore
  * this function, most likely.
  */
-void ast_cc_set_extension_dialable_name(struct ast_channel *chan, const char * const dialable_name);
+void ast_cc_set_extension_dialstring(struct ast_channel *chan, const char * const dialstring);
 
 /*!
  * \since 1.8
@@ -1435,7 +1435,7 @@
  * \param cc_params The CC configuration parameters for the outbound target
  * \param monitor_type The type of monitor to use when CC is requested
  * \param device_name The name of the outbound target device.
- * \param dialable_name The dial string used when calling this specific interface
+ * \param dialstring The dial string used when calling this specific interface
  * \param private_data If a native monitor is being used, and some channel-driver-specific private
  * data has been allocated, then this parameter should contain a pointer to that data. If using a generic
  * monitor, this parameter should remain NULL. Note that if this function should fail at some point,
@@ -1452,7 +1452,7 @@
  * by the channel driver.
  */
 void ast_cc_busy_interface(struct ast_channel *inbound, struct ast_cc_config_params *cc_params,
-	const char *monitor_type, const char * const device_name, const char * const dialable_name, void *private_data);
+	const char *monitor_type, const char * const device_name, const char * const dialstring, void *private_data);
 
 /*!
  * \since 1.8
@@ -1468,7 +1468,7 @@
  * \param cc_params The CC configuration parameters for the outbound target
  * \param monitor_type The type of monitor to use when CC is requested
  * \param device_name The name of the outbound target device.
- * \param dialable_name The dial string used when calling this specific interface
+ * \param dialstring The dial string used when calling this specific interface
  * \param service What kind of CC service is being offered. (CCBS/CCNR/etc...)
  * \param private_data If a native monitor is being used, and some channel-driver-specific private
  * data has been allocated, then this parameter should contain a pointer to that data. If using a generic
@@ -1481,7 +1481,7 @@
  */
 int ast_cc_build_frame(struct ast_channel *chan, struct ast_cc_config_params *cc_params,
 	const char *monitor_type, const char * const device_name,
-	const char * const dialable_name, enum ast_cc_service_type service, void *private_data,
+	const char * const dialstring, enum ast_cc_service_type service, void *private_data,
 	struct ast_frame *frame);
 
 
@@ -1493,7 +1493,7 @@
  * \param cc_params The CC configuration parameters for the outbound target
  * \param monitor_type The type of monitor to use when CC is requested
  * \param device_name The name of the outbound target device.
- * \param dialable_name The dial string used when calling this specific interface
+ * \param dialstring The dial string used when calling this specific interface
  * \param private_data If a native monitor is being used, and some channel-driver-specific private
  * data has been allocated, then this parameter should contain a pointer to that data. If using a generic
  * monitor, this parameter should remain NULL. Note that if this function should fail at some point,
@@ -1507,7 +1507,7 @@
  * \return Nothing
  */
 typedef void (*ast_cc_callback_fn)(struct ast_channel *chan, struct ast_cc_config_params *cc_params,
-	const char *monitor_type, const char * const device_name, const char * const dialable_name, void *private_data);
+	const char *monitor_type, const char * const device_name, const char * const dialstring, void *private_data);
 
 /*!
  * \since 1.8

Modified: team/group/CCSS/main/ccss.c
URL: http://svnview.digium.com/svn/asterisk/team/group/CCSS/main/ccss.c?view=diff&rev=246668&r1=246667&r2=246668
==============================================================================
--- team/group/CCSS/main/ccss.c (original)
+++ team/group/CCSS/main/ccss.c Mon Feb 15 16:08:06 2010
@@ -153,7 +153,7 @@
 	struct ast_cc_config_params config_params;
 	int parent_interface_id;
 	char device_name[AST_CHANNEL_NAME];
-	char dialable_name[AST_CHANNEL_NAME];
+	char dialstring[AST_CHANNEL_NAME];
 };
 
 /*!
@@ -1267,7 +1267,7 @@
 		monitor->callbacks->destructor(monitor->private_data);
 	}
 	cc_unref(monitor->interface, "Unreffing tree's reference to interface");
-	ast_free(monitor->dialable_name);
+	ast_free(monitor->dialstring);
 }
 
 static void cc_interface_tree_destroy(void *data)
@@ -1344,7 +1344,7 @@
 	 * This needs to be preserved as the dialable name on an extension tree item
 	 * which may be created for the local channel.
 	 */
-	char *current_extension_dialable_name;
+	char *current_extension_dialstring;
 	/*!
 	 * Reference-counted "tree" of interfaces.
 	 */
@@ -1367,7 +1367,7 @@
 {
 	struct dialed_cc_interfaces *cc_interfaces = data;
 	cc_unref(cc_interfaces->interface_tree, "Unref dial's ref to interface tree");
-	ast_free(cc_interfaces->current_extension_dialable_name);
+	ast_free(cc_interfaces->current_extension_dialstring);
 	ast_free(cc_interfaces);
 }
 
@@ -1415,7 +1415,7 @@
 	.destroy = dialed_cc_interfaces_destroy,
 };
 
-void ast_cc_set_extension_dialable_name(struct ast_channel *chan, const char * const dialable_name)
+void ast_cc_set_extension_dialstring(struct ast_channel *chan, const char * const dialstring)
 {
 	struct ast_datastore *datastore;
 	struct dialed_cc_interfaces *cc_interfaces;
@@ -1427,7 +1427,7 @@
 	}
 
 	cc_interfaces = datastore->data;
-	cc_interfaces->current_extension_dialable_name = ast_strdup(dialable_name);
+	cc_interfaces->current_extension_dialstring = ast_strdup(dialstring);
 	ast_channel_unlock(chan);
 	return;
 }
@@ -1598,12 +1598,12 @@
  * to happen anyway.
  *
  * \param device_name The name of the device being added to the tree
- * \param dialable_name The dialstring used to dial the device being added
+ * \param dialstring The dialstring used to dial the device being added
  * \param parent_id The parent of this new tree node.
  * \retval NULL Memory allocation failure
  * \retval non-NULL The new ast_cc_interface created.
  */
-static struct ast_cc_monitor *cc_device_monitor_init(const char * const device_name, const char * const dialable_name, const struct ast_control_cc_payload *cc_data, int core_id)
+static struct ast_cc_monitor *cc_device_monitor_init(const char * const device_name, const char * const dialstring, const struct ast_control_cc_payload *cc_data, int core_id)
 {
 	struct ast_cc_interface *cc_interface;
 	struct ast_cc_monitor *monitor;
@@ -1627,7 +1627,7 @@
 		return NULL;
 	}
 
-	if (!(monitor->dialable_name = ast_strdup(dialable_name))) {
+	if (!(monitor->dialstring = ast_strdup(dialstring))) {
 		cc_unref(monitor, "Failed to copy dialable name. Unref monitor");
 		cc_unref(cc_interface, "Failed to copy dialable name");
 		call_destructor_with_no_monitor(cc_data->monitor_type, cc_data->private_data);
@@ -1675,7 +1675,7 @@
 void ast_handle_cc_control_frame(struct ast_channel *inbound, struct ast_channel *outbound, void *frame_data)
 {
 	char *device_name;
-	char *dialable_name;
+	char *dialstring;
 	struct ast_cc_monitor *monitor;
 	struct ast_datastore *cc_datastore;
 	struct dialed_cc_interfaces *cc_interfaces;
@@ -1683,7 +1683,7 @@
 	struct cc_core_instance *core_instance;
 
 	device_name = cc_data->device_name;
-	dialable_name = cc_data->dialable_name;
+	dialstring = cc_data->dialstring;
 
 	ast_channel_lock(inbound);
 	if (!(cc_datastore = ast_channel_datastore_find(inbound, &dialed_cc_interfaces_info, NULL))) {
@@ -1744,7 +1744,7 @@
 	}
 	AST_DLLIST_UNLOCK(cc_interfaces->interface_tree);
 
-	if (!(monitor = cc_device_monitor_init(device_name, dialable_name, cc_data, cc_interfaces->core_id))) {
+	if (!(monitor = cc_device_monitor_init(device_name, dialstring, cc_data, cc_interfaces->core_id))) {
 		ast_log(LOG_WARNING, "Unable to create CC device interface for '%s'. CC services will be unavailable on this interface.\n", device_name);
 		cc_unref(core_instance, "Returning early from ast_handle_cc_control_frame");
 		call_destructor_with_no_monitor(cc_data->monitor_type, cc_data->private_data);
@@ -1833,7 +1833,7 @@
 	/* In situation 2, chan_local will have set the extension's dialable name
 	 * on the datastore, so we need to copy that into this monitor.
 	 */
-	monitor->dialable_name = ast_strdup(interfaces->current_extension_dialable_name);
+	monitor->dialstring = ast_strdup(interfaces->current_extension_dialstring);
 	AST_DLLIST_LOCK(interfaces->interface_tree);
 	cc_ref(monitor, "Interface tree's reference to the monitor");
 	AST_DLLIST_INSERT_TAIL(interfaces->interface_tree, monitor, next);
@@ -2830,7 +2830,7 @@
 	struct cc_recall_ds_data *recall_data;
 	struct ast_str *var_value = ast_str_create(64);
 	int multi = 0;
-	char previous_dialable_name[AST_CHANNEL_NAME] = "";
+	char previous_dialstring[AST_CHANNEL_NAME] = "";
 	int top_level_id;
 
 	ast_channel_lock(chan);
@@ -2848,11 +2848,11 @@
 	top_level_id = monitor_iter->id;
 
 	while ((monitor_iter = AST_DLLIST_NEXT(monitor_iter, next))) {
-		if (monitor_iter->parent_id == top_level_id && strcmp(monitor_iter->dialable_name, previous_dialable_name)) {
-			char *dialable_name = monitor_iter->dialable_name;
-			ast_str_append(&var_value, 0, "%s%s", multi ? "&" : "", dialable_name);
+		if (monitor_iter->parent_id == top_level_id && strcmp(monitor_iter->dialstring, previous_dialstring)) {
+			char *dialstring = monitor_iter->dialstring;
+			ast_str_append(&var_value, 0, "%s%s", multi ? "&" : "", dialstring);
 			multi = 1;
-			ast_copy_string(previous_dialable_name, dialable_name, sizeof(previous_dialable_name));
+			ast_copy_string(previous_dialstring, dialstring, sizeof(previous_dialstring));
 		}
 	}
 	AST_DLLIST_UNLOCK(interface_tree);
@@ -2873,7 +2873,7 @@
 	unsigned int exten_id = 0;
 	struct ast_str *var_value = ast_str_create(64);
 	int multi = 0;
-	char previous_dialable_name[AST_CHANNEL_NAME] = "";
+	char previous_dialstring[AST_CHANNEL_NAME] = "";
 
 	ast_channel_lock(chan);
 	if (!(recall_datastore = ast_channel_datastore_find(chan, &recall_ds_info, NULL))) {
@@ -2907,11 +2907,11 @@
 	/* I kind of feel wrong re-using monitor_iter here, but eh, it works
 	 */
 	while ((monitor_iter = AST_DLLIST_NEXT(monitor_iter, next))) {
-		if (monitor_iter->parent_id == exten_id && strcmp(monitor_iter->dialable_name, previous_dialable_name)) {
-			char *dialable_name = monitor_iter->dialable_name;
-			ast_str_append(&var_value, 0, "%s%s", multi ? "&" : "", dialable_name);
+		if (monitor_iter->parent_id == exten_id && strcmp(monitor_iter->dialstring, previous_dialstring)) {
+			char *dialstring = monitor_iter->dialstring;
+			ast_str_append(&var_value, 0, "%s%s", multi ? "&" : "", dialstring);
 			multi = 1;
-			ast_copy_string(previous_dialable_name, dialable_name, sizeof(previous_dialable_name));
+			ast_copy_string(previous_dialstring, dialstring, sizeof(previous_dialstring));
 		}
 	}
 	AST_DLLIST_UNLOCK(interface_tree);
@@ -3259,7 +3259,7 @@
 }
 
 static int cc_build_payload(struct ast_channel *chan, struct ast_cc_config_params *cc_params,
-	const char *monitor_type, const char * const device_name, const char * dialable_name,
+	const char *monitor_type, const char * const device_name, const char * dialstring,
 	enum ast_cc_service_type service, void *private_data, struct ast_control_cc_payload *payload)
 {
 	struct ast_datastore *datastore;
@@ -3282,12 +3282,12 @@
 	ast_cc_copy_config_params(&payload->config_params, cc_params);
 	payload->parent_interface_id = dial_parent_id;
 	ast_copy_string(payload->device_name, device_name, sizeof(payload->device_name));
-	ast_copy_string(payload->dialable_name, dialable_name, sizeof(payload->dialable_name));
+	ast_copy_string(payload->dialstring, dialstring, sizeof(payload->dialstring));
 	return 0;
 }
 
 int ast_queue_cc_frame(struct ast_channel *chan, const char *monitor_type, 
-		const char * const dialable_name, enum ast_cc_service_type service, void *private_data)
+		const char * const dialstring, enum ast_cc_service_type service, void *private_data)
 {
 	struct ast_frame frame = {0,};
 	char device_name[AST_CHANNEL_NAME];
@@ -3304,7 +3304,7 @@
 		return -1;
 	}
 
-	if (ast_cc_build_frame(chan, cc_params, monitor_type, device_name, dialable_name, service, private_data, &frame)) {
+	if (ast_cc_build_frame(chan, cc_params, monitor_type, device_name, dialstring, service, private_data, &frame)) {
 		/* Frame building failed. We can't use this. */
 		return -1;
 	}
@@ -3315,7 +3315,7 @@
 
 int ast_cc_build_frame(struct ast_channel *chan, struct ast_cc_config_params *cc_params,
 	const char *monitor_type, const char * const device_name,
-	const char * const dialable_name, enum ast_cc_service_type service, void *private_data,
+	const char * const dialstring, enum ast_cc_service_type service, void *private_data,
 	struct ast_frame *frame)
 {
 	struct ast_control_cc_payload *payload = ast_calloc(1, sizeof(*payload));
@@ -3323,7 +3323,7 @@
 	if (!payload) {
 		return -1;
 	}
-	if (cc_build_payload(chan, cc_params, monitor_type, device_name, dialable_name, service, private_data, payload)) {
+	if (cc_build_payload(chan, cc_params, monitor_type, device_name, dialstring, service, private_data, payload)) {
 		/* Something screwed up, we can't make a frame with this */
 		ast_free(payload);
 		return -1;
@@ -3370,10 +3370,10 @@
 }
 
 void ast_cc_busy_interface(struct ast_channel *inbound, struct ast_cc_config_params *cc_params,
-	const char *monitor_type, const char * const device_name, const char * const dialable_name, void *private_data)
+	const char *monitor_type, const char * const device_name, const char * const dialstring, void *private_data)
 {
 	struct ast_control_cc_payload payload;
-	if (cc_build_payload(inbound, cc_params, monitor_type, device_name, dialable_name, AST_CC_CCBS, private_data, &payload)) {
+	if (cc_build_payload(inbound, cc_params, monitor_type, device_name, dialstring, AST_CC_CCBS, private_data, &payload)) {
 		/* Something screwed up. Don't try to handle this payload */
 		call_destructor_with_no_monitor(monitor_type, private_data);
 		return;




More information about the asterisk-commits mailing list