[asterisk-commits] tilghman: trunk r148071 - in /trunk: apps/ channels/ formats/ main/ res/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Thu Oct 9 16:47:03 CDT 2008


Author: tilghman
Date: Thu Oct  9 16:47:02 2008
New Revision: 148071

URL: http://svn.digium.com/view/asterisk?view=rev&rev=148071
Log:
Reverting format addition for now

Modified:
    trunk/apps/app_minivm.c
    trunk/apps/app_voicemail.c
    trunk/channels/chan_agent.c
    trunk/formats/format_wav.c
    trunk/main/file.c
    trunk/res/res_monitor.c

Modified: trunk/apps/app_minivm.c
URL: http://svn.digium.com/view/asterisk/trunk/apps/app_minivm.c?view=diff&rev=148071&r1=148070&r2=148071
==============================================================================
--- trunk/apps/app_minivm.c (original)
+++ trunk/apps/app_minivm.c Thu Oct  9 16:47:02 2008
@@ -917,11 +917,8 @@
 
 	ast_debug(3, "-_-_- Sending mail to %s@%s - Using template %s\n", vmu->username, vmu->domain, template->name);
 
-	if (!strcmp(format, "wav49")) {
+	if (!strcmp(format, "wav49"))
 		format = "WAV";
-	} else if (!strcmp(format, "wav16")) {
-		format = "Wav";
-	}
 
 
 	/* If we have a gain option, process it now with sox */

Modified: trunk/apps/app_voicemail.c
URL: http://svn.digium.com/view/asterisk/trunk/apps/app_voicemail.c?view=diff&rev=148071&r1=148070&r2=148071
==============================================================================
--- trunk/apps/app_voicemail.c (original)
+++ trunk/apps/app_voicemail.c Thu Oct  9 16:47:02 2008
@@ -1755,11 +1755,8 @@
 		tempcopy = 1;
 	}
 
-	if (!strcmp(fmt, "wav49")) {
+	if (!strcmp(fmt, "wav49"))
 		fmt = "WAV";
-	} else if (!strcmp(fmt, "wav16")) {
-		fmt = "Wav";
-	}
 	ast_debug(3, "Storing file '%s', format '%s'\n", fn, fmt);
 
 	/* Make a temporary file instead of piping directly to sendmail, in case the mail
@@ -2756,11 +2753,8 @@
 		c = strchr(fmt, '|');
 		if (c)
 			*c = '\0';
-		if (!strcasecmp(fmt, "wav49")) {
+		if (!strcasecmp(fmt, "wav49"))
 			strcpy(fmt, "WAV");
-		} else if (!strcasecmp(fmt, "wav16")) {
-			strcpy(fmt, "Wav");
-		}
 		snprintf(msgnums, sizeof(msgnums),"%d", msgnum);
 		if (msgnum > -1)
 			make_file(fn, sizeof(fn), dir, msgnum);
@@ -3173,11 +3167,8 @@
 		c = strchr(fmt, '|');
 		if (c)
 			*c = '\0';
-		if (!strcasecmp(fmt, "wav49")) {
+		if (!strcasecmp(fmt, "wav49"))
 			strcpy(fmt, "WAV");
-		} else if (!strcasecmp(fmt, "wav16")) {
-			strcpy(fmt, "Wav");
-		}
 		snprintf(msgnums, sizeof(msgnums),"%d", msgnum);
 		if (msgnum > -1)
 			make_file(fn, sizeof(fn), dir, msgnum);
@@ -3998,11 +3989,8 @@
 		ast_log(AST_LOG_WARNING, "E-mail address missing for mailbox [%s].  E-mail will not be sent.\n", vmu->mailbox);
 		return(0);
 	}
-	if (!strcmp(format, "wav49")) {
+	if (!strcmp(format, "wav49"))
 		format = "WAV";
-	} else if (!strcmp(format, "wav16")) {
-		format = "Wav";
-	}
 	ast_debug(3, "Attaching file '%s', format '%s', uservm is '%d', global is %d\n", attach, format, attach_user_voicemail, ast_test_flag((&globalflags), VM_ATTACH));
 	/* Make a temporary file instead of piping directly to sendmail, in case the mail
 	   command hangs */

Modified: trunk/channels/chan_agent.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_agent.c?view=diff&rev=148071&r1=148070&r2=148071
==============================================================================
--- trunk/channels/chan_agent.c (original)
+++ trunk/channels/chan_agent.c Thu Oct  9 16:47:02 2008
@@ -1201,13 +1201,10 @@
 			recordagentcalls = ast_true(v->value);
 		} else if (!strcasecmp(v->name, "recordformat")) {
 			ast_copy_string(recordformat, v->value, sizeof(recordformat));
-			if (!strcasecmp(v->value, "wav49")) {
+			if (!strcasecmp(v->value, "wav49"))
 				strcpy(recordformatext, "WAV");
-			} else if (!strcasecmp(v->value, "wav16")) {
-				strcpy(recordformatext, "Wav");
-			} else {
+			else
 				ast_copy_string(recordformatext, v->value, sizeof(recordformatext));
-			}
 		} else if (!strcasecmp(v->name, "urlprefix")) {
 			ast_copy_string(urlprefix, v->value, sizeof(urlprefix));
 			if (urlprefix[strlen(urlprefix) - 1] != '/')

Modified: trunk/formats/format_wav.c
URL: http://svn.digium.com/view/asterisk/trunk/formats/format_wav.c?view=diff&rev=148071&r1=148070&r2=148071
==============================================================================
--- trunk/formats/format_wav.c (original)
+++ trunk/formats/format_wav.c Thu Oct  9 16:47:02 2008
@@ -39,7 +39,6 @@
 #define	WAV_BUF_SIZE	320
 
 struct wav_desc {	/* format-specific parameters */
-	int which;
 	int bytes;
 	int lasttimeout;
 	int maxlen;
@@ -71,7 +70,7 @@
 #endif
 
 
-static int check_header(FILE *f, int which)
+static int check_header(FILE *f)
 {
 	int type, size, formtype;
 	int fmt, hsize;
@@ -136,10 +135,7 @@
 		ast_log(LOG_WARNING, "Read failed (freq)\n");
 		return -1;
 	}
-	if (ltohl(freq) != DEFAULT_SAMPLE_RATE && which == 8000) {
-		ast_log(LOG_WARNING, "Unexpected freqency %d\n", ltohl(freq));
-		return -1;
-	} else if (ltohl(freq) != 16000 && which == 16000) {
+	if (ltohl(freq) != DEFAULT_SAMPLE_RATE) {
 		ast_log(LOG_WARNING, "Unexpected freqency %d\n", ltohl(freq));
 		return -1;
 	}
@@ -243,7 +239,7 @@
 	return 0;
 }
 
-static int write_header(FILE *f, int which)
+static int write_header(FILE *f)
 {
 	unsigned int hz=htoll(8000);
 	unsigned int bhz = htoll(16000);
@@ -253,11 +249,6 @@
 	unsigned short bysam = htols(2);
 	unsigned short bisam = htols(16);
 	unsigned int size = htoll(0);
-	if (which == 16000) {
-		hz = htoll(16000);
-		bhz = htoll(32000);
-	}
-
 	/* Write a wav header, ignoring sizes which will be filled in later */
 	fseek(f,0,SEEK_SET);
 	if (fwrite("RIFF", 1, 4, f) != 4) {
@@ -317,16 +308,9 @@
 	   if we did, it would go here.  We also might want to check
 	   and be sure it's a valid file.  */
 	struct wav_desc *tmp = (struct wav_desc *)s->_private;
-	if ((tmp->maxlen = check_header(s->f, tmp->which)) < 0)
+	if ((tmp->maxlen = check_header(s->f)) < 0)
 		return -1;
 	return 0;
-}
-
-static int wav16_open(struct ast_filestream *s)
-{
-	struct wav_desc *tmp = (struct wav_desc *)s->_private;
-	tmp->which = 16000;
-	return wav_open(s);
 }
 
 static int wav_rewrite(struct ast_filestream *s, const char *comment)
@@ -334,18 +318,10 @@
 	/* We don't have any header to read or anything really, but
 	   if we did, it would go here.  We also might want to check
 	   and be sure it's a valid file.  */
-	struct wav_desc *tmp = (struct wav_desc *)s->_private;
-
-	if (write_header(s->f, tmp->which))
+
+	if (write_header(s->f))
 		return -1;
 	return 0;
-}
-
-static int wav16_rewrite(struct ast_filestream *s, const char *comment)
-{
-	struct wav_desc *tmp = (struct wav_desc *)s->_private;
-	tmp->which = 16000;
-	return wav_rewrite(s, comment);
 }
 
 static void wav_close(struct ast_filestream *s)
@@ -374,10 +350,6 @@
 	off_t here;
 	/* Send a frame from the file to the appropriate channel */
 	struct wav_desc *fs = (struct wav_desc *)s->_private;
-
-	if (fs->which == 16000) {
-		bytes *= 2;
-	}
 
 	here = ftello(s->f);
 	if (fs->maxlen - here < bytes)		/* truncate if necessary */
@@ -386,10 +358,10 @@
 		bytes = 0;
 /* 	ast_debug(1, "here: %d, maxlen: %d, bytes: %d\n", here, s->maxlen, bytes); */
 	s->fr.frametype = AST_FRAME_VOICE;
-	s->fr.subclass = fs->which == 16000 ? AST_FORMAT_SLINEAR16 : AST_FORMAT_SLINEAR;
+	s->fr.subclass = AST_FORMAT_SLINEAR;
 	s->fr.mallocd = 0;
 	AST_FRAME_SET_BUFFER(&s->fr, s->buf, AST_FRIENDLY_OFFSET, bytes);
-
+	
 	if ( (res = fread(s->fr.data.ptr, 1, s->fr.datalen, s->f)) <= 0 ) {
 		if (res)
 			ast_log(LOG_WARNING, "Short read (%d) (%s)!\n", res, strerror(errno));
@@ -401,7 +373,7 @@
 	tmp = (short *)(s->fr.data.ptr);
 #if __BYTE_ORDER == __BIG_ENDIAN
 	/* file format is little endian so we need to swap */
-	for( x = 0; x < samples * (fs->which == 16000 ? 2 : 1); x++)
+	for( x = 0; x < samples; x++)
 		tmp[x] = (tmp[x] << 8) | ((tmp[x] & 0xff00) >> 8);
 #endif
 
@@ -413,7 +385,7 @@
 {
 #if __BYTE_ORDER == __BIG_ENDIAN
 	int x;
-	short tmp[16000], *tmpi;
+	short tmp[8000], *tmpi;
 #endif
 	struct wav_desc *s = (struct wav_desc *)fs->_private;
 	int res;
@@ -422,11 +394,8 @@
 		ast_log(LOG_WARNING, "Asked to write non-voice frame!\n");
 		return -1;
 	}
-	if (f->subclass != AST_FORMAT_SLINEAR && s->which == 0) {
+	if (f->subclass != AST_FORMAT_SLINEAR) {
 		ast_log(LOG_WARNING, "Asked to write non-SLINEAR frame (%d)!\n", f->subclass);
-		return -1;
-	} else if (f->subclass != AST_FORMAT_SLINEAR16 && s->which == 16000) {
-		ast_log(LOG_WARNING, "Asked to write non-SLINEAR16 frame (%d)!\n", f->subclass);
 		return -1;
 	}
 	if (!f->datalen)
@@ -452,15 +421,9 @@
 	}
 
 	s->bytes += f->datalen;
-
+		
 	return 0;
-}
-
-static int wav16_write(struct ast_filestream *fs, struct ast_frame *f)
-{
-	struct wav_desc *s = (struct wav_desc *)fs->_private;
-	s->which = 16000;
-	return wav_write(fs, f);
+
 }
 
 static int wav_seek(struct ast_filestream *fs, off_t sample_offset, int whence)
@@ -517,38 +480,16 @@
 	.desc_size = sizeof(struct wav_desc),
 };
 
-static const struct ast_format Wav_f = {
-	.name = "wav16",
-	.exts = "Wav|wav16",
-	.format = AST_FORMAT_SLINEAR16,
-	.open =	wav16_open,
-	.rewrite = wav16_rewrite,
-	.write = wav16_write,
-	.seek = wav_seek,
-	.trunc = wav_trunc,
-	.tell =	wav_tell,
-	.read = wav_read,
-	.close = wav_close,
-	.buf_size = WAV_BUF_SIZE * 2 + AST_FRIENDLY_OFFSET,
-	.desc_size = sizeof(struct wav_desc),
-};
-
 static int load_module(void)
 {
 	if (ast_format_register(&wav_f))
 		return AST_MODULE_LOAD_FAILURE;
-	ast_format_register(&Wav_f);
 	return AST_MODULE_LOAD_SUCCESS;
 }
 
 static int unload_module(void)
 {
-	int res;
-	if (!(res = ast_format_unregister(Wav_f.name))) {
-		return ast_format_unregister(wav_f.name);
-	} else {
-		return res;
-	}
+	return ast_format_unregister(wav_f.name);
 }	
 
 AST_MODULE_INFO_STANDARD(ASTERISK_GPL_KEY, "Microsoft WAV format (8000Hz Signed Linear)");

Modified: trunk/main/file.c
URL: http://svn.digium.com/view/asterisk/trunk/main/file.c?view=diff&rev=148071&r1=148070&r2=148071
==============================================================================
--- trunk/main/file.c (original)
+++ trunk/main/file.c Thu Oct  9 16:47:02 2008
@@ -248,8 +248,6 @@
 
 	if (!strcmp(ext, "wav49"))
 		ext = "WAV";
-	if (!strcmp(ext, "wav16"))
-		ext = "Wav";
 
 	if (filename[0] == '/')
 		asprintf(&fn, "%s.%s", filename, ext);

Modified: trunk/res/res_monitor.c
URL: http://svn.digium.com/view/asterisk/trunk/res/res_monitor.c?view=diff&rev=148071&r1=148070&r2=148071
==============================================================================
--- trunk/res/res_monitor.c (original)
+++ trunk/res/res_monitor.c Thu Oct  9 16:47:02 2008
@@ -324,7 +324,7 @@
 		if (chan->monitor->joinfiles && !ast_strlen_zero(chan->monitor->filename_base)) {
 			char tmp[1024];
 			char tmp2[1024];
-			const char *format = !strcasecmp(chan->monitor->format,"wav49") ? "WAV" : !strcasecmp(chan->monitor->format, "wav16") ? "Wav" : chan->monitor->format;
+			const char *format = !strcasecmp(chan->monitor->format,"wav49") ? "WAV" : chan->monitor->format;
 			char *name = chan->monitor->filename_base;
 			int directory = strchr(name, '/') ? 1 : 0;
 			const char *dir = directory ? "" : ast_config_AST_MONITOR_DIR;




More information about the asterisk-commits mailing list