Avoid cppcheck warnings; removing unused vars and a bit of cleanup.
[asterisk/asterisk.git] / formats / format_h263.c
index b0b5cb2..ec08867 100644 (file)
  * \ingroup formats
  * \arg See \ref AstVideo
  */
+
+/*** MODULEINFO
+       <support_level>core</support_level>
+ ***/
  
 #include "asterisk.h"
 
@@ -52,9 +56,8 @@ struct h263_desc {
 static int h263_open(struct ast_filestream *s)
 {
        unsigned int ts;
-       int res;
 
-       if ((res = fread(&ts, 1, sizeof(ts), s->f)) < sizeof(ts)) {
+       if (fread(&ts, 1, sizeof(ts), s->f) < sizeof(ts)) {
                ast_log(LOG_WARNING, "Empty file!\n");
                return -1;
        }
@@ -64,7 +67,7 @@ static int h263_open(struct ast_filestream *s)
 static struct ast_frame *h263_read(struct ast_filestream *s, int *whennext)
 {
        int res;
-       int mark;
+       uint32_t mark;
        unsigned short len;
        unsigned int ts;
        struct h263_desc *fs = (struct h263_desc *)s->_private;
@@ -80,17 +83,19 @@ static struct ast_frame *h263_read(struct ast_filestream *s, int *whennext)
                return NULL;
        }
        s->fr.frametype = AST_FRAME_VIDEO;
-       s->fr.subclass = AST_FORMAT_H263;
+       ast_format_set(&s->fr.subclass.format, AST_FORMAT_H263, 0);
        s->fr.mallocd = 0;
        AST_FRAME_SET_BUFFER(&s->fr, s->buf, AST_FRIENDLY_OFFSET, len);
-       if ((res = fread(s->fr.data, 1, s->fr.datalen, s->f)) != s->fr.datalen) {
+       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));
                return NULL;
        }
        s->fr.samples = fs->lastts;     /* XXX what ? */
        s->fr.datalen = len;
-       s->fr.subclass |= mark;
+       if (mark) {
+               ast_format_set_video_mark(&s->fr.subclass.format);
+       }
        s->fr.delivery.tv_sec = 0;
        s->fr.delivery.tv_usec = 0;
        if ((res = fread(&ts, 1, sizeof(ts), s->f)) == sizeof(ts)) {
@@ -106,18 +111,14 @@ static int h263_write(struct ast_filestream *fs, struct ast_frame *f)
        int res;
        unsigned int ts;
        unsigned short len;
-       int subclass;
-       int mark=0;
+       uint32_t mark = 0;
        if (f->frametype != AST_FRAME_VIDEO) {
                ast_log(LOG_WARNING, "Asked to write non-video frame!\n");
                return -1;
        }
-       subclass = f->subclass;
-       if (subclass & 0x1)
-               mark=0x8000;
-       subclass &= ~0x1;
-       if (subclass != AST_FORMAT_H263) {
-               ast_log(LOG_WARNING, "Asked to write non-h263 frame (%d)!\n", f->subclass);
+       mark = ast_format_get_video_mark(&f->subclass.format) ? 0x8000 : 0;
+       if (f->subclass.format.id != AST_FORMAT_H263) {
+               ast_log(LOG_WARNING, "Asked to write non-h263 frame (%s)!\n", ast_getformatname(&f->subclass.format));
                return -1;
        }
        ts = htonl(f->samples);
@@ -130,7 +131,7 @@ static int h263_write(struct ast_filestream *fs, struct ast_frame *f)
                        ast_log(LOG_WARNING, "Bad write (%d/2): %s\n", res, strerror(errno));
                        return -1;
        }
-       if ((res = fwrite(f->data, 1, f->datalen, fs->f)) != f->datalen) {
+       if ((res = fwrite(f->data.ptr, 1, f->datalen, fs->f)) != f->datalen) {
                        ast_log(LOG_WARNING, "Bad write (%d/%d): %s\n", res, f->datalen, strerror(errno));
                        return -1;
        }
@@ -145,10 +146,19 @@ static int h263_seek(struct ast_filestream *fs, off_t sample_offset, int whence)
 
 static int h263_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 h263 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 h263 filestream %p: %s\n", fs, strerror(errno));
+               return -1;
+       }
+       /* Truncate file to current length */
+       return ftruncate(fd, cur);
 }
 
 static off_t h263_tell(struct ast_filestream *fs)
@@ -157,10 +167,9 @@ static off_t h263_tell(struct ast_filestream *fs)
        return offset;  /* XXX totally bogus, needs fixing */
 }
 
-static const struct ast_format h263_f = {
+static struct ast_format_def h263_f = {
        .name = "h263",
        .exts = "h263",
-       .format = AST_FORMAT_H263,
        .open = h263_open,
        .write = h263_write,
        .seek = h263_seek,
@@ -173,14 +182,19 @@ static const struct ast_format h263_f = {
 
 static int load_module(void)
 {
-       if (ast_format_register(&h263_f))
+       ast_format_set(&h263_f.format, AST_FORMAT_H263, 0);
+       if (ast_format_def_register(&h263_f))
                return AST_MODULE_LOAD_FAILURE;
        return AST_MODULE_LOAD_SUCCESS;
 }
 
 static int unload_module(void)
 {
-       return ast_format_unregister(h263_f.name);
-}      
+       return ast_format_def_unregister(h263_f.name);
+}
 
-AST_MODULE_INFO_STANDARD(ASTERISK_GPL_KEY, "Raw H.263 data");
+AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_LOAD_ORDER, "Raw H.263 data",
+       .load = load_module,
+       .unload = unload_module,
+       .load_pri = AST_MODPRI_APP_DEPEND
+);