[svn-commits] oej: trunk r286271 - /trunk/channels/chan_sip.c
SVN commits to the Digium repositories
svn-commits at lists.digium.com
Sat Sep 11 12:10:58 CDT 2010
Author: oej
Date: Sat Sep 11 12:10:54 2010
New Revision: 286271
URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=286271
Log:
Formatting changes.
Modified:
trunk/channels/chan_sip.c
Modified: trunk/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/trunk/channels/chan_sip.c?view=diff&rev=286271&r1=286270&r2=286271
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Sat Sep 11 12:10:54 2010
@@ -6520,8 +6520,9 @@
ast_debug(3, "*** Joint capabilities are %s \n", ast_getformatname_multiple(buf, SIPBUFSIZE, i->jointcapability));
ast_debug(3, "*** Our capabilities are %s \n", ast_getformatname_multiple(buf, SIPBUFSIZE, i->capability));
ast_debug(3, "*** AST_CODEC_CHOOSE formats are %s \n", ast_getformatname_multiple(buf, SIPBUFSIZE, ast_codec_choose(&i->prefs, what, 1)));
- if (i->prefcodec)
+ if (i->prefcodec) {
ast_debug(3, "*** Our preferred formats from the incoming channel are %s \n", ast_getformatname_multiple(buf, SIPBUFSIZE, i->prefcodec));
+ }
/* XXX Why are we choosing a codec from the native formats?? */
fmt = ast_best_codec(tmp->nativeformats);
@@ -6546,10 +6547,11 @@
needtext = i->jointcapability & AST_FORMAT_TEXT_MASK; /* Inbound call */
}
- if (needvideo)
+ if (needvideo) {
ast_debug(3, "This channel can handle video! HOLLYWOOD next!\n");
- else
+ } else {
ast_debug(3, "This channel will not be able to handle video.\n");
+ }
enable_dsp_detect(i);
@@ -6573,13 +6575,16 @@
ast_channel_set_fd(tmp, 2, ast_rtp_instance_fd(i->vrtp, 0));
ast_channel_set_fd(tmp, 3, ast_rtp_instance_fd(i->vrtp, 1));
}
- if (needtext && i->trtp)
+ if (needtext && i->trtp) {
ast_channel_set_fd(tmp, 4, ast_rtp_instance_fd(i->trtp, 0));
- if (i->udptl)
+ }
+ if (i->udptl) {
ast_channel_set_fd(tmp, 5, ast_udptl_fd(i->udptl));
-
- if (state == AST_STATE_RING)
+ }
+
+ if (state == AST_STATE_RING) {
tmp->rings = 1;
+ }
tmp->adsicpe = AST_ADSI_UNAVAILABLE;
tmp->writeformat = fmt;
@@ -6596,14 +6601,18 @@
tmp->pickupgroup = i->pickupgroup;
tmp->caller.id.name.presentation = i->callingpres;
tmp->caller.id.number.presentation = i->callingpres;
- if (!ast_strlen_zero(i->parkinglot))
+ if (!ast_strlen_zero(i->parkinglot)) {
ast_string_field_set(tmp, parkinglot, i->parkinglot);
- if (!ast_strlen_zero(i->accountcode))
+ }
+ if (!ast_strlen_zero(i->accountcode)) {
ast_string_field_set(tmp, accountcode, i->accountcode);
- if (i->amaflags)
+ }
+ if (i->amaflags) {
tmp->amaflags = i->amaflags;
- if (!ast_strlen_zero(i->language))
+ }
+ if (!ast_strlen_zero(i->language)) {
ast_string_field_set(tmp, language, i->language);
+ }
i->owner = tmp;
ast_module_ref(ast_module_info->self);
ast_copy_string(tmp->context, i->context, sizeof(tmp->context));
@@ -6631,14 +6640,18 @@
}
tmp->priority = 1;
- if (!ast_strlen_zero(i->uri))
+ if (!ast_strlen_zero(i->uri)) {
pbx_builtin_setvar_helper(tmp, "SIPURI", i->uri);
- if (!ast_strlen_zero(i->domain))
+ }
+ if (!ast_strlen_zero(i->domain)) {
pbx_builtin_setvar_helper(tmp, "SIPDOMAIN", i->domain);
- if (!ast_strlen_zero(i->callid))
+ }
+ if (!ast_strlen_zero(i->callid)) {
pbx_builtin_setvar_helper(tmp, "SIPCALLID", i->callid);
- if (i->rtp)
+ }
+ if (i->rtp) {
ast_jb_configure(tmp, &global_jbconf);
+ }
/* Set channel variables for this call from configuration */
for (v = i->chanvars ; v ; v = v->next) {
@@ -6653,14 +6666,16 @@
tmp = NULL;
}
- if (i->do_history)
+ if (i->do_history) {
append_history(i, "NewChan", "Channel %s - from %s", tmp->name, i->callid);
+ }
/* Inform manager user about new channel and their SIP call ID */
- if (sip_cfg.callevents)
+ if (sip_cfg.callevents) {
manager_event(EVENT_FLAG_SYSTEM, "ChannelUpdate",
"Channel: %s\r\nUniqueid: %s\r\nChanneltype: %s\r\nSIPcallid: %s\r\nSIPfullcontact: %s\r\n",
tmp->name, tmp->uniqueid, "SIP", i->callid, i->fullcontact);
+ }
return tmp;
}
@@ -6668,8 +6683,9 @@
/*! \brief Reads one line of SIP message body */
static char *get_body_by_line(const char *line, const char *name, int nameLen, char delimiter)
{
- if (!strncasecmp(line, name, nameLen) && line[nameLen] == delimiter)
+ if (!strncasecmp(line, name, nameLen) && line[nameLen] == delimiter) {
return ast_skip_blanks(line + nameLen + 1);
+ }
return "";
}
@@ -6729,8 +6745,9 @@
for (x = 0; x < req->lines; x++) {
r = get_body_by_line(REQ_OFFSET_TO_STR(req, line[x]), name, len, delimiter);
- if (r[0] != '\0')
+ if (r[0] != '\0') {
return r;
+ }
}
return "";
@@ -6794,8 +6811,9 @@
const char *header = REQ_OFFSET_TO_STR(req, header[x]);
if (!strncasecmp(header, name, len)) {
const char *r = header + len; /* skip name */
- if (sip_cfg.pedanticsipchecking)
+ if (sip_cfg.pedanticsipchecking) {
r = ast_skip_blanks(r);
+ }
if (*r == ':') {
*start = x+1;
@@ -6803,8 +6821,9 @@
}
}
}
- if (pass == 0) /* Try aliases */
+ if (pass == 0) { /* Try aliases */
name = find_alias(name, NULL);
+ }
}
/* Don't return NULL, so get_header is always a valid pointer */
@@ -6849,11 +6868,13 @@
if (sipdebug_text) {
int i;
unsigned char* arr = f->data.ptr;
- for (i=0; i < f->datalen; i++)
+ for (i=0; i < f->datalen; i++) {
ast_verbose("%c", (arr[i] > ' ' && arr[i] < '}') ? arr[i] : '.');
+ }
ast_verbose(" -> ");
- for (i=0; i < f->datalen; i++)
+ for (i=0; i < f->datalen; i++) {
ast_verbose("%02X ", arr[i]);
+ }
ast_verbose("\n");
}
break;
@@ -6871,8 +6892,9 @@
}
/* We already hold the channel lock */
- if (!p->owner || (f && f->frametype != AST_FRAME_VOICE))
+ if (!p->owner || (f && f->frametype != AST_FRAME_VOICE)) {
return f;
+ }
if (f && f->subclass.codec != (p->owner->nativeformats & AST_FORMAT_AUDIO_MASK)) {
if (!(f->subclass.codec & p->jointcapability)) {
More information about the svn-commits
mailing list