formats/format_wav: enhancing log message "Not a wav file" to be clear on what is...
[asterisk/asterisk.git] / formats / format_g723.c
index 594f3d1..c3c194d 100644 (file)
  * \arg Extensions: g723, g723sf
  * \ingroup formats
  */
-#include <unistd.h>
-#include <netinet/in.h>
-#include <arpa/inet.h>
-#include <stdlib.h>
-#include <stdio.h>
-#include <errno.h>
-#include <string.h>
-#include <sys/time.h>
 
+/*** MODULEINFO
+       <support_level>core</support_level>
+ ***/
 #include "asterisk.h"
 
 ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 
-#include "asterisk/lock.h"
-#include "asterisk/channel.h"
-#include "asterisk/file.h"
-#include "asterisk/logger.h"
-#include "asterisk/sched.h"
+#include "asterisk/mod_format.h"
 #include "asterisk/module.h"
 
 #define G723_MAX_SIZE 1024
@@ -74,10 +65,10 @@ static struct ast_frame *g723_read(struct ast_filestream *s, int *whennext)
        }
        /* Read the data into the buffer */
        s->fr.frametype = AST_FRAME_VOICE;
-       s->fr.subclass = AST_FORMAT_G723_1;
+       ast_format_set(&s->fr.subclass.format, AST_FORMAT_G723_1, 0);
        s->fr.mallocd = 0;
-       FR_SET_BUF(&s->fr, s->buf, AST_FRIENDLY_OFFSET, size);
-       if ((res = fread(s->fr.data, 1, s->fr.datalen, s->f)) != size) {
+       AST_FRAME_SET_BUFFER(&s->fr, s->buf, AST_FRIENDLY_OFFSET, size);
+       if ((res = fread(s->fr.data.ptr, 1, s->fr.datalen, s->f)) != size) {
                ast_log(LOG_WARNING, "Short read (%d of %d bytes) (%s)!\n", res, size, strerror(errno));
                return NULL;
        }
@@ -87,15 +78,15 @@ static struct ast_frame *g723_read(struct ast_filestream *s, int *whennext)
 
 static int g723_write(struct ast_filestream *s, struct ast_frame *f)
 {
-       u_int32_t delay;
-       u_int16_t size;
+       uint32_t delay;
+       uint16_t size;
        int res;
        /* XXX there used to be a check s->fr means a read stream */
        if (f->frametype != AST_FRAME_VOICE) {
                ast_log(LOG_WARNING, "Asked to write non-voice frame!\n");
                return -1;
        }
-       if (f->subclass != AST_FORMAT_G723_1) {
+       if (f->subclass.format.id != AST_FORMAT_G723_1) {
                ast_log(LOG_WARNING, "Asked to write non-g723 frame!\n");
                return -1;
        }
@@ -113,7 +104,7 @@ static int g723_write(struct ast_filestream *s, struct ast_frame *f)
                ast_log(LOG_WARNING, "Unable to write size: res=%d (%s)\n", res, strerror(errno));
                return -1;
        }
-       if ((res = fwrite(f->data, 1, f->datalen, s->f)) != f->datalen) {
+       if ((res = fwrite(f->data.ptr, 1, f->datalen, s->f)) != f->datalen) {
                ast_log(LOG_WARNING, "Unable to write frame: res=%d (%s)\n", res, strerror(errno));
                return -1;
        }       
@@ -127,10 +118,19 @@ static int g723_seek(struct ast_filestream *fs, off_t sample_offset, int whence)
 
 static int g723_trunc(struct ast_filestream *fs)
 {
-       /* Truncate file to current length */
-       if (ftruncate(fileno(fs->f), ftello(fs->f)) < 0)
+       int fd;
+       off_t cur;
+
+       if ((fd = fileno(fs->f)) < 0) {
+               ast_log(AST_LOG_WARNING, "Unable to determine file descriptor for g723 filestream %p: %s\n", fs, strerror(errno));
                return -1;
-       return 0;
+       }
+       if ((cur = ftello(fs->f)) < 0) {
+               ast_log(AST_LOG_WARNING, "Unable to determine current position in g723 filestream %p: %s\n", fs, strerror(errno));
+               return -1;
+       }
+       /* Truncate file to current length */
+       return ftruncate(fd, cur);
 }
 
 static off_t g723_tell(struct ast_filestream *fs)
@@ -138,42 +138,33 @@ static off_t g723_tell(struct ast_filestream *fs)
        return -1;
 }
 
-static struct ast_format_lock me = { .usecnt = -1 };
-
-static const struct ast_format g723_1_f = {
+static struct ast_format_def g723_1_f = {
        .name = "g723sf",
        .exts = "g723|g723sf",
-       .format = AST_FORMAT_G723_1,
        .write = g723_write,
        .seek = g723_seek,
        .trunc = g723_trunc,
        .tell = g723_tell,
        .read = g723_read,
        .buf_size = G723_MAX_SIZE + AST_FRIENDLY_OFFSET,
-       .lockp = &me,
 };
 
-int load_module()
-{
-       return ast_format_register(&g723_1_f);
-}
-
-int unload_module()
+static int load_module(void)
 {
-       return ast_format_unregister(g723_1_f.name);
-}      
+       ast_format_set(&g723_1_f.format, AST_FORMAT_G723_1, 0);
 
-int usecount()
-{
-       return me.usecnt;
+       if (ast_format_def_register(&g723_1_f))
+               return AST_MODULE_LOAD_FAILURE;
+       return AST_MODULE_LOAD_SUCCESS;
 }
 
-const char *description()
+static int unload_module(void)
 {
-       return "G.723.1 Simple Timestamp File Format";
+       return ast_format_def_unregister(g723_1_f.name);
 }
 
-const char *key()
-{
-       return ASTERISK_GPL_KEY;
-}
+AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_LOAD_ORDER, "G.723.1 Simple Timestamp File Format",
+       .load = load_module,
+       .unload = unload_module,
+       .load_pri = AST_MODPRI_APP_DEPEND
+);