test_json: Fix test names with reserved words
[asterisk/asterisk.git] / formats / format_ilbc.c
index e18c78a..8b41ab2 100644 (file)
 
 #include "asterisk.h"
 
-ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
-
 #include "asterisk/mod_format.h"
 #include "asterisk/module.h"
 #include "asterisk/endian.h"
+#include "asterisk/format_cache.h"
 
 /* Some Ideas for this code came from makeg729e.c by Jeffrey Chilton */
 
@@ -48,13 +47,18 @@ static struct ast_frame *ilbc_read(struct ast_filestream *s, int *whennext)
 {
        int res;
        /* Send a frame from the file to the appropriate channel */
-       s->fr.frametype = AST_FRAME_VOICE;
-       ast_format_set(&s->fr.subclass.format, AST_FORMAT_ILBC, 0);
-       s->fr.mallocd = 0;
        AST_FRAME_SET_BUFFER(&s->fr, s->buf, AST_FRIENDLY_OFFSET, ILBC_BUF_SIZE);
        if ((res = fread(s->fr.data.ptr, 1, s->fr.datalen, s->f)) != s->fr.datalen) {
-               if (res)
-                       ast_log(LOG_WARNING, "Short read (%d) (%s)!\n", res, strerror(errno));
+               if (feof(s->f)) {
+                       if (res) {
+                               ast_debug(3, "Incomplete frame data at end of %s file "
+                                                 "(expected %d bytes, read %d)\n",
+                                                 ast_format_get_name(s->fr.subclass.format), s->fr.datalen, res);
+                       }
+               } else {
+                       ast_log(LOG_ERROR, "Error while reading %s file: %s\n",
+                                       ast_format_get_name(s->fr.subclass.format), strerror(errno));
+               }
                return NULL;
        }
        *whennext = s->fr.samples = ILBC_SAMPLES;
@@ -64,14 +68,6 @@ static struct ast_frame *ilbc_read(struct ast_filestream *s, int *whennext)
 static int ilbc_write(struct ast_filestream *fs, struct ast_frame *f)
 {
        int res;
-       if (f->frametype != AST_FRAME_VOICE) {
-               ast_log(LOG_WARNING, "Asked to write non-voice frame!\n");
-               return -1;
-       }
-       if (f->subclass.format.id != AST_FORMAT_ILBC) {
-               ast_log(LOG_WARNING, "Asked to write non-iLBC frame (%s)!\n", ast_getformatname(&f->subclass.format));
-               return -1;
-       }
        if (f->datalen % 50) {
                ast_log(LOG_WARNING, "Invalid data length, %d, should be multiple of 50\n", f->datalen);
                return -1;
@@ -118,7 +114,7 @@ static int ilbc_trunc(struct ast_filestream *fs)
                ast_log(AST_LOG_WARNING, "Unable to determine file descriptor for iLBC filestream %p: %s\n", fs, strerror(errno));
                return -1;
        }
-       if ((cur = ftello(fs->f) < 0)) {
+       if ((cur = ftello(fs->f)) < 0) {
                ast_log(AST_LOG_WARNING, "Unable to determine current position in iLBC filestream %p: %s\n", fs, strerror(errno));
                return -1;
        }
@@ -145,9 +141,9 @@ static struct ast_format_def ilbc_f = {
 
 static int load_module(void)
 {
-       ast_format_set(&ilbc_f.format, AST_FORMAT_ILBC, 0);
+       ilbc_f.format = ast_format_ilbc;
        if (ast_format_def_register(&ilbc_f))
-               return AST_MODULE_LOAD_FAILURE;
+               return AST_MODULE_LOAD_DECLINE;
        return AST_MODULE_LOAD_SUCCESS;
 }
 
@@ -157,6 +153,7 @@ static int unload_module(void)
 }
 
 AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_LOAD_ORDER, "Raw iLBC data",
+       .support_level = AST_MODULE_SUPPORT_CORE,
        .load = load_module,
        .unload = unload_module,
        .load_pri = AST_MODPRI_APP_DEPEND