[libpri-commits] rmudgett: branch dvossel/aoc_send r1657 - /team/dvossel/aoc_send/

SVN commits to the libpri project libpri-commits at lists.digium.com
Thu Apr 22 13:25:42 CDT 2010


Author: rmudgett
Date: Thu Apr 22 13:25:38 2010
New Revision: 1657

URL: http://svnview.digium.com/svn/libpri?view=rev&rev=1657
Log:
Cosmetic changes.  Blank lines, doxygen, curly braces.

Modified:
    team/dvossel/aoc_send/libpri.h
    team/dvossel/aoc_send/pri_aoc.c
    team/dvossel/aoc_send/pri_facility.c
    team/dvossel/aoc_send/pri_facility.h
    team/dvossel/aoc_send/pri_internal.h

Modified: team/dvossel/aoc_send/libpri.h
URL: http://svnview.digium.com/svn/libpri/team/dvossel/aoc_send/libpri.h?view=diff&rev=1657&r1=1656&r2=1657
==============================================================================
--- team/dvossel/aoc_send/libpri.h (original)
+++ team/dvossel/aoc_send/libpri.h Thu Apr 22 13:25:38 2010
@@ -871,7 +871,6 @@
 };
 
 enum PRI_AOC_REQ_RSP {
-
 	/* Error Results */
 	PRI_AOC_REQ_RSP_ERROR_NOT_IMPLEMENTED,
 	PRI_AOC_REQ_RSP_ERROR_NOT_AVAILABLE,
@@ -893,7 +892,6 @@
 };
 
 struct pri_subcmd_aoc_request_response {
-
 	/*!
 	 * \brief aoc_s data from response
 	 */

Modified: team/dvossel/aoc_send/pri_aoc.c
URL: http://svnview.digium.com/svn/libpri/team/dvossel/aoc_send/pri_aoc.c?view=diff&rev=1657&r1=1656&r2=1657
==============================================================================
--- team/dvossel/aoc_send/pri_aoc.c (original)
+++ team/dvossel/aoc_send/pri_aoc.c Thu Apr 22 13:25:38 2010
@@ -687,9 +687,9 @@
 
 /*!
  * \internal
- * \brief Determine the etsi aoc-e billing_id value from the subcmd.
- *
- * \param PRI_AOC_E_BILLING_ID billing_id
+ * \brief Determine the ETSI AOC-E billing_id value from the subcmd.
+ *
+ * \param billing_id from upper layer.
  *
  * \retval -1 failure
  * \retval etsi billing id
@@ -698,21 +698,21 @@
 {
 	switch (billing_id) {
 	case PRI_AOC_E_BILLING_ID_NORMAL:
-		return 0;
+		return 0;/* normalCharging */
 	case PRI_AOC_E_BILLING_ID_REVERSE:
-		return 1;
+		return 1;/* reverseCharging */
 	case PRI_AOC_E_BILLING_ID_CREDIT_CARD:
-		return 2;
+		return 2;/* creditCardCharging */
 	case PRI_AOC_E_BILLING_ID_CALL_FORWARDING_UNCONDITIONAL:
-		return 3;
+		return 3;/* callForwardingUnconditional */
 	case PRI_AOC_E_BILLING_ID_CALL_FORWARDING_BUSY:
-		return 4;
+		return 4;/* callForwardingBusy */
 	case PRI_AOC_E_BILLING_ID_CALL_FORWARDING_NO_REPLY:
-		return 5;
+		return 5;/* callForwardingNoReply */
 	case PRI_AOC_E_BILLING_ID_CALL_DEFLECTION:
-		return 6;
+		return 6;/* callDeflection */
 	case PRI_AOC_E_BILLING_ID_CALL_TRANSFER:
-		return 7;
+		return 7;/* callTransfer */
 	case PRI_AOC_E_BILLING_ID_NOT_AVAILABLE:
 		break;
 	}
@@ -722,9 +722,9 @@
 
 /*!
  * \internal
- * \brief Determine the etsi aoc-d billing_id value from the subcmd.
- *
- * \param PRI_AOC_D_BILLING_ID billing_id
+ * \brief Determine the ETSI AOC-D billing_id value from the subcmd.
+ *
+ * \param billing_id from upper layer.
  *
  * \retval -1 failure
  * \retval etsi billing id
@@ -733,11 +733,11 @@
 {
 	switch (billing_id) {
 	case PRI_AOC_D_BILLING_ID_NORMAL:
-		return 0;
+		return 0;/* normalCharging */
 	case PRI_AOC_D_BILLING_ID_REVERSE:
-		return 1;
+		return 1;/* reverseCharging */
 	case PRI_AOC_D_BILLING_ID_CREDIT_CARD:
-		return 2;
+		return 2;/* creditCardCharging */
 	case PRI_AOC_D_BILLING_ID_NOT_AVAILABLE:
 		break;
 	}
@@ -879,7 +879,7 @@
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param pos Starting position to encode the facility ie contents.
  * \param end End of facility ie contents encoding data buffer.
- * \param aoc_e, the aoc-e data to encode.
+ * \param aoc_e the AOC-E data to encode.
  *
  * \retval Start of the next ASN.1 component to encode on success.
  * \retval NULL on error.
@@ -900,14 +900,14 @@
 	msg.invoke_id = get_invokeid(ctrl);
 
 	if (aoc_e->charge == PRI_AOC_DE_CHARGE_FREE) {
-		msg.args.etsi.AOCECurrency.type = 1;	/* currency info */
+		msg.args.etsi.AOCECurrency.type = 1;	/* currency_info */
 		msg.args.etsi.AOCECurrency.currency_info.free_of_charge = 1;
 	} else if ((aoc_e->charge == PRI_AOC_DE_CHARGE_CURRENCY) && (aoc_e->recorded.money.amount.cost >= 0)) {
-		msg.args.etsi.AOCECurrency.type = 1;	/* currency info */
+		msg.args.etsi.AOCECurrency.type = 1;	/* currency_info */
 		aoc_enc_etsi_subcmd_recorded_currency(&aoc_e->recorded.money,
 			&msg.args.etsi.AOCECurrency.currency_info.specific.recorded);
 	} else {
-		msg.args.etsi.AOCECurrency.type = 0;	/* not_available */
+		msg.args.etsi.AOCECurrency.type = 0;	/* charge_not_available */
 	}
 
 	if (aoc_subcmd_aoc_e_etsi_billing_id(aoc_e->billing_id) != -1) {
@@ -919,7 +919,7 @@
 	switch (aoc_e->associated.charging_type) {
 	case PRI_AOC_E_CHARGING_ASSOCIATION_NUMBER:
 		msg.args.etsi.AOCECurrency.currency_info.charging_association_present = 1;
-		msg.args.etsi.AOCECurrency.currency_info.charging_association.type = 1; /* number */
+		msg.args.etsi.AOCECurrency.currency_info.charging_association.type = 1; /* charged_number */
 		pri_copy_party_number_to_q931(&q931_number, &aoc_e->associated.charge.number);
 		q931_copy_number_to_rose(ctrl,
 			&msg.args.etsi.AOCECurrency.currency_info.charging_association.number,
@@ -927,7 +927,7 @@
 		break;
 	case PRI_AOC_E_CHARGING_ASSOCIATION_ID:
 		msg.args.etsi.AOCECurrency.currency_info.charging_association_present = 1;
-		msg.args.etsi.AOCECurrency.currency_info.charging_association.type = 0; /* identifier */
+		msg.args.etsi.AOCECurrency.currency_info.charging_association.type = 0; /* charge_identifier */
 		msg.args.etsi.AOCECurrency.currency_info.charging_association.id =
 			aoc_e->associated.charge.id;
 		break;
@@ -948,7 +948,7 @@
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param pos Starting position to encode the facility ie contents.
  * \param end End of facility ie contents encoding data buffer.
- * \param aoc_e, the aoc-e data to encode.
+ * \param aoc_e the AOC-E data to encode.
  *
  * \retval Start of the next ASN.1 component to encode on success.
  * \retval NULL on error.
@@ -977,7 +977,7 @@
 		aoc_enc_etsi_subcmd_recorded_units(&aoc_e->recorded.unit,
 			&msg.args.etsi.AOCEChargingUnit.charging_unit.specific.recorded);
 	} else {
-		msg.args.etsi.AOCEChargingUnit.type = 0;	/* not_available */
+		msg.args.etsi.AOCEChargingUnit.type = 0;	/* charge_not_available */
 	}
 
 	if (aoc_subcmd_aoc_e_etsi_billing_id(aoc_e->billing_id) != -1) {
@@ -989,7 +989,7 @@
 	switch (aoc_e->associated.charging_type) {
 	case PRI_AOC_E_CHARGING_ASSOCIATION_NUMBER:
 		msg.args.etsi.AOCEChargingUnit.charging_unit.charging_association_present = 1;
-		msg.args.etsi.AOCEChargingUnit.charging_unit.charging_association.type = 1; /* number */
+		msg.args.etsi.AOCEChargingUnit.charging_unit.charging_association.type = 1; /* charged_number */
 		pri_copy_party_number_to_q931(&q931_number, &aoc_e->associated.charge.number);
 		q931_copy_number_to_rose(ctrl,
 			&msg.args.etsi.AOCEChargingUnit.charging_unit.charging_association.number,
@@ -997,7 +997,7 @@
 		break;
 	case PRI_AOC_E_CHARGING_ASSOCIATION_ID:
 		msg.args.etsi.AOCEChargingUnit.charging_unit.charging_association_present = 1;
-		msg.args.etsi.AOCEChargingUnit.charging_unit.charging_association.type = 0; /* identifier */
+		msg.args.etsi.AOCEChargingUnit.charging_unit.charging_association.type = 0; /* charge_identifier */
 		msg.args.etsi.AOCEChargingUnit.charging_unit.charging_association.id =
 			aoc_e->associated.charge.id;
 		break;
@@ -1018,7 +1018,7 @@
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param pos Starting position to encode the facility ie contents.
  * \param end End of facility ie contents encoding data buffer.
- * \param aoc_d, the aoc-d data to encode.
+ * \param aoc_d the AOC-D data to encode.
  *
  * \retval Start of the next ASN.1 component to encode on success.
  * \retval NULL on error.
@@ -1038,13 +1038,13 @@
 	msg.invoke_id = get_invokeid(ctrl);
 
 	if (aoc_d->charge == PRI_AOC_DE_CHARGE_FREE) {
-		msg.args.etsi.AOCDChargingUnit.type = 1;	/* free of charge */
+		msg.args.etsi.AOCDChargingUnit.type = 1;	/* free_of_charge */
 	} else if ((aoc_d->charge == PRI_AOC_DE_CHARGE_UNITS) &&  (aoc_d->recorded.unit.num_items > 0)) {
-		msg.args.etsi.AOCDChargingUnit.type = 2;	/* charging_unit */
+		msg.args.etsi.AOCDChargingUnit.type = 2;	/* specific_charging_units */
 		aoc_enc_etsi_subcmd_recorded_units(&aoc_d->recorded.unit,
 			&msg.args.etsi.AOCDChargingUnit.specific.recorded);
 	} else {
-		msg.args.etsi.AOCDChargingUnit.type = 0;	/* not_available */
+		msg.args.etsi.AOCDChargingUnit.type = 0;	/* charge_not_available */
 	}
 
 	if (aoc_subcmd_aoc_d_etsi_billing_id(aoc_d->billing_id) != -1) {
@@ -1053,7 +1053,6 @@
 			aoc_subcmd_aoc_d_etsi_billing_id(aoc_d->billing_id);
 	}
 
-
 	pos = rose_encode_invoke(ctrl, pos, end, &msg);
 
 	return pos;
@@ -1066,7 +1065,7 @@
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param pos Starting position to encode the facility ie contents.
  * \param end End of facility ie contents encoding data buffer.
- * \param aoc_d, the aoc-d data to encode.
+ * \param aoc_d the AOC-D data to encode.
  *
  * \retval Start of the next ASN.1 component to encode on success.
  * \retval NULL on error.
@@ -1086,13 +1085,13 @@
 	msg.invoke_id = get_invokeid(ctrl);
 
 	if (aoc_d->charge == PRI_AOC_DE_CHARGE_FREE) {
-		msg.args.etsi.AOCDCurrency.type = 1;	/* free of charge */
+		msg.args.etsi.AOCDCurrency.type = 1;	/* free_of_charge */
 	} else if ((aoc_d->charge == PRI_AOC_DE_CHARGE_CURRENCY) && (aoc_d->recorded.money.amount.cost >= 0)) {
-		msg.args.etsi.AOCDCurrency.type = 2;	/* specific currency */
+		msg.args.etsi.AOCDCurrency.type = 2;	/* specific_currency */
 		aoc_enc_etsi_subcmd_recorded_currency(&aoc_d->recorded.money,
 			&msg.args.etsi.AOCDCurrency.specific.recorded);
 	} else {
-		msg.args.etsi.AOCDCurrency.type = 0;	/* not_available */
+		msg.args.etsi.AOCDCurrency.type = 0;	/* charge_not_available */
 	}
 
 	if (aoc_subcmd_aoc_d_etsi_billing_id(aoc_d->billing_id) != -1) {
@@ -1113,7 +1112,7 @@
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param pos Starting position to encode the facility ie contents.
  * \param end End of facility ie contents encoding data buffer.
- * \param aoc_s, the aoc-s data to encode.
+ * \param aoc_s the AOC-S data to encode.
  *
  * \retval Start of the next ASN.1 component to encode on success.
  * \retval NULL on error.
@@ -1133,9 +1132,9 @@
 	msg.invoke_id = get_invokeid(ctrl);
 
 	if (!aoc_s->num_items || (aoc_s->item[0].rate_type != PRI_AOC_RATE_TYPE_SPECIAL_CODE)) {
-		msg.args.etsi.AOCSSpecialArr.type = 0;
+		msg.args.etsi.AOCSSpecialArr.type = 0;/* charge_not_available */
 	} else {
-		msg.args.etsi.AOCSSpecialArr.type = 1;
+		msg.args.etsi.AOCSSpecialArr.type = 1;/* special_arrangement_info */
 		msg.args.etsi.AOCSSpecialArr.special_arrangement = aoc_s->item[0].rate.special;
 	}
 
@@ -1151,7 +1150,7 @@
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param pos Starting position to encode the facility ie contents.
  * \param end End of facility ie contents encoding data buffer.
- * \param aoc_s, the aoc-s data to encode.
+ * \param aoc_s the AOC-S data to encode.
  *
  * \retval Start of the next ASN.1 component to encode on success.
  * \retval NULL on error.
@@ -1171,10 +1170,10 @@
 	msg.invoke_id = get_invokeid(ctrl);
 
 	if (aoc_s->num_items) {
-		msg.args.etsi.AOCSCurrency.type = 1; /* charge list is present */
+		msg.args.etsi.AOCSCurrency.type = 1; /* currency_info_list */
 		enc_etsi_subcmd_aoc_s_currency_info(aoc_s, &msg.args.etsi.AOCSCurrency.currency_info);
 	} else {
-		msg.args.etsi.AOCSCurrency.type = 0; /* charge not available */
+		msg.args.etsi.AOCSCurrency.type = 0; /* charge_not_available */
 	}
 
 	pos = rose_encode_invoke(ctrl, pos, end, &msg);
@@ -1189,21 +1188,22 @@
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param pos Starting position to encode the facility ie contents.
  * \param end End of facility ie contents encoding data buffer.
- * \param response, the response to the request
- * \param invoke_id, the request's invoke id
- * \param aoc_s, the rate list associated with a response to AOC-S request
+ * \param response the response to the request
+ * \param invoke_id the request's invoke id
+ * \param aoc_s the rate list associated with a response to AOC-S request
+ *    Could be NULL.
  *
  * \retval Start of the next ASN.1 component to encode on success.
  * \retval NULL on error.
  */
 static unsigned char *enc_etsi_aoc_request_response(struct pri *ctrl, unsigned char *pos,
-	unsigned char *end, int response, int invoke_id, const struct pri_subcmd_aoc_s *aoc_s)
+	unsigned char *end, enum PRI_AOC_REQ_RSP response, int invoke_id, const struct pri_subcmd_aoc_s *aoc_s)
 {
 	struct rose_msg_result msg_result = { 0, };
 	struct rose_msg_error msg_error = { 0, };
 	int is_error = 0;
+
 	pos = facility_encode_header(ctrl, pos, end, NULL);
-
 	if (!pos) {
 		return NULL;
 	}
@@ -1214,17 +1214,17 @@
 			return NULL;
 		}
 		enc_etsi_subcmd_aoc_s_currency_info(aoc_s, &msg_result.args.etsi.ChargingRequest.u.currency_info);
-		msg_result.args.etsi.ChargingRequest.type = 0;
+		msg_result.args.etsi.ChargingRequest.type = 0;/* currency_info_list */
 		break;
 	case PRI_AOC_REQ_RSP_SPECIAL_ARR:
 		if (!aoc_s) {
 			return NULL;
 		}
-		msg_result.args.etsi.ChargingRequest.type = 1;
+		msg_result.args.etsi.ChargingRequest.type = 1;/* special_arrangement_info */
 		msg_result.args.etsi.ChargingRequest.u.special_arrangement = aoc_s->item[0].rate.special;
 		break;
 	case PRI_AOC_REQ_RSP_CHARGING_INFO_FOLLOWS:
-		msg_result.args.etsi.ChargingRequest.type = 2;
+		msg_result.args.etsi.ChargingRequest.type = 2;/* charging_info_follows */
 		break;
 	case PRI_AOC_REQ_RSP_ERROR_NOT_IMPLEMENTED:
 		msg_error.code = ROSE_ERROR_Gen_NotImplemented;
@@ -1258,13 +1258,13 @@
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param pos Starting position to encode the facility ie contents.
  * \param end End of facility ie contents encoding data buffer.
- * \param aoc_request, the aoc charging request data to encode.
+ * \param aoc_request the aoc charging request data to encode.
  *
  * \retval Start of the next ASN.1 component to encode on success.
  * \retval NULL on error.
  */
 static unsigned char *enc_etsi_aoc_request(struct pri *ctrl, unsigned char *pos,
-	unsigned char *end, int request)
+	unsigned char *end, enum PRI_AOC_REQUEST request)
 {
 	struct rose_msg_invoke msg;
 
@@ -1279,13 +1279,13 @@
 
 	switch (request) {
 	case PRI_AOC_REQUEST_S:
-		msg.args.etsi.ChargingRequest.charging_case = 0;
+		msg.args.etsi.ChargingRequest.charging_case = 0;/* chargingInformationAtCallSetup */
 		break;
 	case PRI_AOC_REQUEST_D:
-		msg.args.etsi.ChargingRequest.charging_case = 1;
+		msg.args.etsi.ChargingRequest.charging_case = 1;/* chargingDuringACall */
 		break;
 	case PRI_AOC_REQUEST_E:
-		msg.args.etsi.ChargingRequest.charging_case = 2;
+		msg.args.etsi.ChargingRequest.charging_case = 2;/* chargingAtTheEndOfACall */
 		break;
 	default:
 		/* no valid request parameters are present */
@@ -1303,7 +1303,7 @@
  *
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param call Call leg from which to encode AOC.
- * \param invoke_id
+ * \param invoke_id the request's invoke id
  * \param aoc_s Optional AOC-S rate list for response
  *
  * \note if aoc_s is NULL, then a response will be sent back as AOC-S not available.
@@ -1324,7 +1324,6 @@
 	}
 
 	end = enc_etsi_aoc_request_response(ctrl, buffer, buffer + sizeof(buffer), response, invoke_id, aoc_s);
-
 	if (!end) {
 		return -1;
 	}
@@ -1340,26 +1339,24 @@
 	return 0;
 }
 
-
 /*!
  * \internal
  * \brief Send the ETSI AOC Request Response message for AOC-D and AOC-E requests
  *
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param call Call leg from which to encode AOC.
- * \param response code
- * \param invoke_id
+ * \param response the response to the request
+ * \param invoke_id the request's invoke id
  *
  * \retval 0 on success.
  * \retval -1 on error.
  */
-static int aoc_de_request_response_encode(struct pri *ctrl, q931_call *call, int response, int invoke_id)
+static int aoc_de_request_response_encode(struct pri *ctrl, q931_call *call, enum PRI_AOC_REQ_RSP response, int invoke_id)
 {
 	unsigned char buffer[255];
 	unsigned char *end = 0;
 
 	end = enc_etsi_aoc_request_response(ctrl, buffer, buffer + sizeof(buffer), response, invoke_id, NULL);
-
 	if (!end) {
 		return -1;
 	}
@@ -1397,7 +1394,6 @@
 	}
 
 	subcmd = q931_alloc_subcommand(ctrl);
-
 	if (!subcmd) {
 		return 1;
 	}
@@ -1465,12 +1461,12 @@
  *
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param call Call leg from which to encode AOC.
- * \param aoc_request, the aoc charging request payload data to encode.
+ * \param aoc_request the aoc charging request payload data to encode.
  *
  * \retval 0 on success.
  * \retval -1 on error.
  */
-static int aoc_charging_request_encode(struct pri *ctrl, q931_call *call, int request)
+static int aoc_charging_request_encode(struct pri *ctrl, q931_call *call, enum PRI_AOC_REQUEST request)
 {
 	unsigned char buffer[255];
 	unsigned char *end = 0;
@@ -1494,14 +1490,13 @@
 	return pri_call_apdu_queue(call, Q931_SETUP, buffer, end - buffer, &response);
 }
 
-
 /*!
  * \internal
  * \brief Send the ETSI AOCS invoke message.
  *
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param call Call leg from which to encode AOC.
- * \param aoc_s, the AOC-S payload data to encode.
+ * \param aoc_s the AOC-S payload data to encode.
  *
  * \retval 0 on success.
  * \retval -1 on error.
@@ -1516,7 +1511,6 @@
 	} else {
 		end = enc_etsi_aocs_currency(ctrl, buffer, buffer + sizeof(buffer), aoc_s);
 	}
-
 	if (!end) {
 		return -1;
 	}
@@ -1538,7 +1532,7 @@
  *
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param call Call leg from which to encode AOC.
- * \param aoc_d, the aoc_d payload data to encode.
+ * \param aoc_d the AOC-D payload data to encode.
  *
  * \retval 0 on success.
  * \retval -1 on error.
@@ -1546,8 +1540,7 @@
 static int aoc_d_encode(struct pri *ctrl, q931_call *call, const struct pri_subcmd_aoc_d *aoc_d)
 {
 	unsigned char buffer[255];
-	unsigned char *end = 0;
-
+	unsigned char *end = NULL;
 
 	switch (aoc_d->charge) {
 	case PRI_AOC_DE_CHARGE_NOT_AVAILABLE:
@@ -1559,7 +1552,6 @@
 		end = enc_etsi_aocd_charging_unit(ctrl, buffer, buffer + sizeof(buffer), aoc_d);
 		break;
 	}
-
 	if (!end) {
 		return -1;
 	}
@@ -1581,7 +1573,7 @@
  *
  * \param ctrl D channel controller for diagnostic messages or global options.
  * \param call Call leg from which to encode AOC.
- * \param aoc_e, the aoc_e payload data to encode.
+ * \param aoc_e the AOC-E payload data to encode.
  *
  * \retval 0 on success.
  * \retval -1 on error.
@@ -1589,8 +1581,7 @@
 static int aoc_e_encode(struct pri *ctrl, q931_call *call, const struct pri_subcmd_aoc_e *aoc_e)
 {
 	unsigned char buffer[255];
-	unsigned char *end = 0;
-
+	unsigned char *end = NULL;
 
 	switch (aoc_e->charge) {
 	case PRI_AOC_DE_CHARGE_NOT_AVAILABLE:
@@ -1602,7 +1593,6 @@
 		end = enc_etsi_aoce_charging_unit(ctrl, buffer, buffer + sizeof(buffer), aoc_e);
 		break;
 	}
-
 	if (!end) {
 		return -1;
 	}
@@ -1617,8 +1607,9 @@
 
 int pri_aoc_de_request_response_send(struct pri *ctrl, q931_call *call, int response, int invoke_id)
 {
-	if (!ctrl || !call)
-		return -1;
+	if (!ctrl || !call) {
+		return -1;
+	}
 
 	switch (ctrl->switchtype) {
 	case PRI_SWITCH_EUROISDN_E1:
@@ -1635,8 +1626,9 @@
 
 int pri_aoc_s_request_response_send(struct pri *ctrl, q931_call *call, int invoke_id, const struct pri_subcmd_aoc_s *aoc_s)
 {
-	if (!ctrl || !call)
-		return -1;
+	if (!ctrl || !call) {
+		return -1;
+	}
 
 	switch (ctrl->switchtype) {
 	case PRI_SWITCH_EUROISDN_E1:
@@ -1651,7 +1643,17 @@
 	return 0;
 }
 
-int pri_aoc_charging_request_send(struct pri *ctrl, q931_call *call, enum PRI_AOC_REQUEST aoc_request_flag)
+/*!
+ * \brief Send AOC request message.
+ *
+ * \param ctrl D channel controller.
+ * \param call Q.931 call leg.
+ * \param aoc types to request
+ *
+ * \retval 0 on success
+ * \retval -1 on failure
+ */
+int aoc_charging_request_send(struct pri *ctrl, q931_call *call, enum PRI_AOC_REQUEST aoc_request_flag)
 {
 	int res;
 
@@ -1687,8 +1689,9 @@
 
 int pri_aoc_s_send(struct pri *ctrl, q931_call *call, const struct pri_subcmd_aoc_s *aoc_s)
 {
-	if (!ctrl || !call)
-		return -1;
+	if (!ctrl || !call) {
+		return -1;
+	}
 
 	switch (ctrl->switchtype) {
 	case PRI_SWITCH_EUROISDN_E1:
@@ -1705,8 +1708,9 @@
 
 int pri_aoc_d_send(struct pri *ctrl, q931_call *call, const struct pri_subcmd_aoc_d *aoc_d)
 {
-	if (!ctrl || !call)
-		return -1;
+	if (!ctrl || !call) {
+		return -1;
+	}
 
 	switch (ctrl->switchtype) {
 	case PRI_SWITCH_EUROISDN_E1:
@@ -1722,8 +1726,9 @@
 
 int pri_aoc_e_send(struct pri *ctrl, q931_call *call, const struct pri_subcmd_aoc_e *aoc_e)
 {
-	if (!ctrl || !call)
-		return -1;
+	if (!ctrl || !call) {
+		return -1;
+	}
 
 	switch (ctrl->switchtype) {
 	case PRI_SWITCH_EUROISDN_E1:
@@ -1740,7 +1745,6 @@
 
 int pri_sr_set_aoc_charging_request(struct pri_sr *sr, int charging_request)
 {
-
 	if (charging_request & PRI_AOC_REQUEST_S) {
 		sr->aoc_charging_request |= PRI_AOC_REQUEST_S;
 	}
@@ -1752,7 +1756,7 @@
 	}
 
 	return 0;
-
-}
+}
+
 /* ------------------------------------------------------------------- */
 /* end pri_aoc.c */

Modified: team/dvossel/aoc_send/pri_facility.c
URL: http://svnview.digium.com/svn/libpri/team/dvossel/aoc_send/pri_facility.c?view=diff&rev=1657&r1=1656&r2=1657
==============================================================================
--- team/dvossel/aoc_send/pri_facility.c (original)
+++ team/dvossel/aoc_send/pri_facility.c Thu Apr 22 13:25:38 2010
@@ -3157,8 +3157,7 @@
 	case PRI_SWITCH_EUROISDN_E1:
 	case PRI_SWITCH_EUROISDN_T1:
 		if (call->aoc_charging_request) {
-
-			pri_aoc_charging_request_send(ctrl, call, call->aoc_charging_request);
+			aoc_charging_request_send(ctrl, call, call->aoc_charging_request);
 		}
 		if (PTMP_MODE(ctrl)) {
 			/* PTMP mode */

Modified: team/dvossel/aoc_send/pri_facility.h
URL: http://svnview.digium.com/svn/libpri/team/dvossel/aoc_send/pri_facility.h?view=diff&rev=1657&r1=1656&r2=1657
==============================================================================
--- team/dvossel/aoc_send/pri_facility.h (original)
+++ team/dvossel/aoc_send/pri_facility.h Thu Apr 22 13:25:38 2010
@@ -251,6 +251,7 @@
 void pri_cc_qsig_cancel(struct pri *ctrl, q931_call *call, int msgtype, const struct rose_msg_invoke *invoke);
 void pri_cc_qsig_exec_possible(struct pri *ctrl, q931_call *call, int msgtype, const struct rose_msg_invoke *invoke);
 
+int aoc_charging_request_send(struct pri *ctrl, q931_call *c, enum PRI_AOC_REQUEST aoc_request_flag);
 void aoc_etsi_aoc_request(struct pri *ctrl, q931_call *call, const struct rose_msg_invoke *invoke);
 void aoc_etsi_aoc_s_currency(struct pri *ctrl, const struct rose_msg_invoke *invoke);
 void aoc_etsi_aoc_s_special_arrangement(struct pri *ctrl, const struct rose_msg_invoke *invoke);

Modified: team/dvossel/aoc_send/pri_internal.h
URL: http://svnview.digium.com/svn/libpri/team/dvossel/aoc_send/pri_internal.h?view=diff&rev=1657&r1=1656&r2=1657
==============================================================================
--- team/dvossel/aoc_send/pri_internal.h (original)
+++ team/dvossel/aoc_send/pri_internal.h Thu Apr 22 13:25:38 2010
@@ -962,18 +962,6 @@
 void q931_cc_indirect(struct pri *ctrl, struct pri_cc_record *cc_record, void (*func)(struct pri *ctrl, q931_call *call, struct pri_cc_record *cc_record));
 
 /*!
- * \brief Send AOC request message.
- *
- * \param ctrl D channel controller.
- * \param call Q.931 call leg.
- * \param aoc types to request
- *
- * \retval 0 on success
- * \retval -1 on failure
- */
-int pri_aoc_charging_request_send(struct pri *ctrl, q931_call *c, enum PRI_AOC_REQUEST aoc_request_flag);
-
-/*!
  * \brief Get the master PRI control structure.
  *
  * \param ctrl D channel controller.




More information about the libpri-commits mailing list