瀏覽代碼

sca: removed non-documented modparam 'outbound_proxy'

- issue reported by oej, GH #580.
  Modparam outbound_proxy was defined in sca.c and sca.h but never used
  in the module.
Kevin Scott Adams 9 年之前
父節點
當前提交
4a50b819fc
共有 2 個文件被更改,包括 54 次插入51 次删除
  1. 54 50
      modules/sca/sca.c
  2. 0 1
      modules/sca/sca.h

+ 54 - 50
modules/sca/sca.c

@@ -67,38 +67,44 @@ static void sca_mod_destroy(void);
 static int sca_set_config(sca_mod *);
 static int sca_set_config(sca_mod *);
 
 
 /* EXPORTED COMMANDS */
 /* EXPORTED COMMANDS */
-static cmd_export_t cmds[] = { { "sca_handle_subscribe", sca_handle_subscribe,
-		0, NULL, REQUEST_ROUTE }, { "sca_call_info_update",
-		sca_call_info_update, 0, NULL,
-		REQUEST_ROUTE | FAILURE_ROUTE | ONREPLY_ROUTE }, {
-		"sca_call_info_update", sca_call_info_update, 1, fixup_var_int_1,
-		REQUEST_ROUTE | FAILURE_ROUTE | ONREPLY_ROUTE },
-		{ NULL, NULL, -1, 0, 0 }, };
+static cmd_export_t cmds[] = {
+		{ "sca_handle_subscribe", sca_handle_subscribe,
+				0, NULL, REQUEST_ROUTE },
+		{ "sca_call_info_update", sca_call_info_update,
+				0, NULL, REQUEST_ROUTE | FAILURE_ROUTE | ONREPLY_ROUTE },
+		{"sca_call_info_update", sca_call_info_update,
+				1, fixup_var_int_1, REQUEST_ROUTE | FAILURE_ROUTE | ONREPLY_ROUTE },
+		{ NULL, NULL, -1, 0, 0 },
+};
 
 
 /* EXPORTED RPC INTERFACE */
 /* EXPORTED RPC INTERFACE */
-static rpc_export_t sca_rpc[] = { { "sca.all_subscriptions",
-		sca_rpc_show_all_subscriptions, sca_rpc_show_all_subscriptions_doc, 0 },
+static rpc_export_t sca_rpc[] = {
+		{ "sca.all_subscriptions", sca_rpc_show_all_subscriptions,
+				sca_rpc_show_all_subscriptions_doc, 0 },
 		{ "sca.subscription_count", sca_rpc_subscription_count,
 		{ "sca.subscription_count", sca_rpc_subscription_count,
-				sca_rpc_subscription_count_doc, 0 }, { "sca.show_subscription",
-				sca_rpc_show_subscription, sca_rpc_show_subscription_doc, 0 }, {
-				"sca.subscribers", sca_rpc_show_subscribers,
-				sca_rpc_show_subscribers_doc, 0 }, {
-				"sca.deactivate_all_subscriptions",
-				sca_rpc_deactivate_all_subscriptions,
-				sca_rpc_deactivate_all_subscriptions_doc, 0 }, {
-				"sca.deactivate_subscription", sca_rpc_deactivate_subscription,
-				sca_rpc_deactivate_subscription_doc, 0 }, {
-				"sca.all_appearances", sca_rpc_show_all_appearances,
-				sca_rpc_show_all_appearances_doc, 0 }, { "sca.show_appearance",
-				sca_rpc_show_appearance, sca_rpc_show_appearance_doc, 0 }, {
-				"sca.seize_appearance", sca_rpc_seize_appearance,
-				sca_rpc_seize_appearance_doc, 0 }, { "sca.update_appearance",
-				sca_rpc_update_appearance, sca_rpc_update_appearance_doc, 0 }, {
-				"sca.release_appearance", sca_rpc_release_appearance,
-				sca_rpc_release_appearance_doc, 0 }, { NULL, NULL, NULL, 0 }, };
+				sca_rpc_subscription_count_doc, 0 },
+		{ "sca.show_subscription", sca_rpc_show_subscription,
+				sca_rpc_show_subscription_doc, 0 },
+		{"sca.subscribers", sca_rpc_show_subscribers,
+				sca_rpc_show_subscribers_doc, 0 },
+		{"sca.deactivate_all_subscriptions", sca_rpc_deactivate_all_subscriptions,
+				sca_rpc_deactivate_all_subscriptions_doc, 0 },
+		{"sca.deactivate_subscription", sca_rpc_deactivate_subscription,
+				sca_rpc_deactivate_subscription_doc, 0 },
+		{"sca.all_appearances", sca_rpc_show_all_appearances,
+				sca_rpc_show_all_appearances_doc, 0 },
+		{ "sca.show_appearance", sca_rpc_show_appearance,
+				sca_rpc_show_appearance_doc, 0 },
+		{"sca.seize_appearance", sca_rpc_seize_appearance,
+				sca_rpc_seize_appearance_doc, 0 },
+		{ "sca.update_appearance", sca_rpc_update_appearance,
+				sca_rpc_update_appearance_doc, 0 },
+		{"sca.release_appearance", sca_rpc_release_appearance,
+				sca_rpc_release_appearance_doc, 0 },
+		{ NULL, NULL, NULL, 0 },
+};
 
 
 /* EXPORTED PARAMETERS */
 /* EXPORTED PARAMETERS */
-str outbound_proxy = STR_NULL;
 str db_url = STR_STATIC_INIT( DEFAULT_DB_URL );
 str db_url = STR_STATIC_INIT( DEFAULT_DB_URL );
 str db_subs_table = STR_STATIC_INIT( "sca_subscriptions" );
 str db_subs_table = STR_STATIC_INIT( "sca_subscriptions" );
 str db_state_table = STR_STATIC_INIT( "sca_state" );
 str db_state_table = STR_STATIC_INIT( "sca_state" );
@@ -108,27 +114,29 @@ int call_info_max_expires = 3600;
 int line_seize_max_expires = 15;
 int line_seize_max_expires = 15;
 int purge_expired_interval = 120;
 int purge_expired_interval = 120;
 
 
-static param_export_t params[] = { { "outbound_proxy", PARAM_STR,
-		&outbound_proxy }, { "db_url", PARAM_STR, &db_url }, { "subs_table",
-		PARAM_STR, &db_subs_table },
-		{ "state_table", PARAM_STR, &db_state_table }, { "db_update_interval",
-				INT_PARAM, &db_update_interval }, { "hash_table_size",
-				INT_PARAM, &hash_table_size }, { "call_info_max_expires",
-				INT_PARAM, &call_info_max_expires }, { "line_seize_max_expires",
-				INT_PARAM, &line_seize_max_expires }, {
-				"purge_expired_interval", INT_PARAM, &purge_expired_interval },
-		{ NULL, 0, NULL }, };
+static param_export_t params[] = {
+		{ "db_url", PARAM_STR, &db_url },
+		{ "subs_table", PARAM_STR, &db_subs_table },
+		{ "state_table", PARAM_STR, &db_state_table },
+		{ "db_update_interval", INT_PARAM, &db_update_interval },
+		{ "hash_table_size", INT_PARAM, &hash_table_size },
+		{ "call_info_max_expires", INT_PARAM, &call_info_max_expires },
+		{ "line_seize_max_expires", INT_PARAM, &line_seize_max_expires },
+		{"purge_expired_interval", INT_PARAM, &purge_expired_interval },
+		{ NULL, 0, NULL },
+};
 
 
 /* MODULE EXPORTS */
 /* MODULE EXPORTS */
-struct module_exports exports = { "sca", /* module name */
-cmds, /* exported functions */
-NULL, /* RPC methods */
-params, /* exported parameters */
-sca_mod_init, /* module initialization function */
-NULL, /* response handling function */
-sca_mod_destroy, /* destructor function */
-NULL, /* oncancel function */
-sca_child_init, /* per-child initialization function */
+struct module_exports exports = {
+		"sca", /* module name */
+		cmds, /* exported functions */
+		NULL, /* RPC methods */
+		params, /* exported parameters */
+		sca_mod_init, /* module initialization function */
+		NULL, /* response handling function */
+		sca_mod_destroy, /* destructor function */
+		NULL, /* oncancel function */
+		sca_child_init, /* per-child initialization function */
 };
 };
 
 
 static int sca_bind_sl(sca_mod *scam, sl_api_t *sl_api) {
 static int sca_bind_sl(sca_mod *scam, sl_api_t *sl_api) {
@@ -207,10 +215,6 @@ static int sca_set_config(sca_mod *scam) {
 	}
 	}
 	memset(scam->cfg, 0, sizeof(sca_config));
 	memset(scam->cfg, 0, sizeof(sca_config));
 
 
-	if (outbound_proxy.s) {
-		scam->cfg->outbound_proxy = &outbound_proxy;
-	}
-
 	if (!db_url.s || db_url.len <= 0) {
 	if (!db_url.s || db_url.len <= 0) {
 		LM_ERR( "sca_set_config: db_url must be set!\n" );
 		LM_ERR( "sca_set_config: db_url must be set!\n" );
 		return (-1);
 		return (-1);

+ 0 - 1
modules/sca/sca.h

@@ -28,7 +28,6 @@
 #define SCA_H
 #define SCA_H
 
 
 struct _sca_config {
 struct _sca_config {
-	str *outbound_proxy;
 	str *db_url;
 	str *db_url;
 	str *subs_table;
 	str *subs_table;
 	str *state_table;
 	str *state_table;