Merge changes from team/russell/g722-sillyness ...
authorRussell Bryant <russell@russellbryant.com>
Fri, 7 Mar 2008 00:24:58 +0000 (00:24 +0000)
committerRussell Bryant <russell@russellbryant.com>
Fri, 7 Mar 2008 00:24:58 +0000 (00:24 +0000)
Fix a number of other places where the number of samples in a G722 frame was
not properly handled because of various reasons.

main/rtp.c:
 - When a G722 frame is read from the smoother, the number of samples in the
   frame must be divided by 2 before being sent out over the network.  Even
   though G722 is 16 kHz, an error in some previous spec has made it so that
   we have to list the number of samples such as if it was 8 kHz.

main/file.c:
 - When scheduling the next time to expect a frame, take into account that the
   format of the file we're reading from may not be 8 kHz.

codecs/codec_g722.c:
 - When converting from G722 to slinear, g722_decode() expects its samples
   parameter to be in the silly (real samples / 2) format.  Make it so.
 - When converting from slinear to G722, properly set the number of samples in
   the frame to be the number of bytes of output * 2.

formats/format_pcm.c:
 - This format module handles G722, among a number of other formats.  However,
   the read() and seek() functions did not account for the fact that G722 has
   2 samples per byte.

(closes issue #12130, reported by rickross, patched by me)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@106501 65c4cc65-6c06-0410-ace0-fbb531ad65f3

codecs/codec_g722.c
formats/format_pcm.c
main/file.c
main/rtp.c

index 99c3856..7dd2744 100644 (file)
@@ -102,9 +102,13 @@ static int g722tolin_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
 {
        struct g722_decoder_pvt *tmp = pvt->pvt;
        int out_samples;
+       int in_samples;
+
+       /* g722_decode expects the samples to be in the invalid samples / 2 format */
+       in_samples = f->samples / 2;
 
        out_samples = g722_decode(&tmp->g722, (int16_t *) &pvt->outbuf[pvt->samples * sizeof(int16_t)], 
-               (uint8_t *) f->data, f->samples);
+               (uint8_t *) f->data, in_samples);
 
        pvt->samples += out_samples;
 
@@ -121,7 +125,7 @@ static int lintog722_framein(struct ast_trans_pvt *pvt, struct ast_frame *f)
        outlen = g722_encode(&tmp->g722, (uint8_t *) (&pvt->outbuf[pvt->datalen]), 
                (int16_t *) f->data, f->samples);
 
-       pvt->samples += outlen;
+       pvt->samples += outlen * 2;
 
        pvt->datalen += outlen;
 
index c514b58..1183535 100644 (file)
@@ -89,7 +89,10 @@ static struct ast_frame *pcm_read(struct ast_filestream *s, int *whennext)
                return NULL;
        }
        s->fr.datalen = res;
-       *whennext = s->fr.samples = res;
+       if (s->fmt->format == AST_FORMAT_G722)
+               *whennext = s->fr.samples = res * 2;
+       else
+               *whennext = s->fr.samples = res;
        return &s->fr;
 }
 
@@ -367,24 +370,32 @@ static int au_rewrite(struct ast_filestream *s, const char *comment)
 static int au_seek(struct ast_filestream *fs, off_t sample_offset, int whence)
 {
        off_t min, max, cur;
-       long offset = 0, samples;
-       
-       samples = sample_offset;
+       long offset = 0, bytes;
+
+       if (fs->fmt->format == AST_FORMAT_G722)
+               bytes = sample_offset / 2;
+       else
+               bytes = sample_offset;
+
        min = AU_HEADER_SIZE;
        cur = ftello(fs->f);
        fseek(fs->f, 0, SEEK_END);
        max = ftello(fs->f);
+
        if (whence == SEEK_SET)
-               offset = samples + min;
+               offset = bytes + min;
        else if (whence == SEEK_CUR || whence == SEEK_FORCECUR)
-               offset = samples + cur;
+               offset = bytes + cur;
        else if (whence == SEEK_END)
-               offset = max - samples;
-        if (whence != SEEK_FORCECUR) {
+               offset = max - bytes;
+
+       if (whence != SEEK_FORCECUR) {
                offset = (offset > max) ? max : offset;
        }
+
        /* always protect the header space. */
        offset = (offset < min) ? min : offset;
+
        return fseeko(fs->f, offset, SEEK_SET);
 }
 
index fb68475..1bec4fc 100644 (file)
@@ -669,7 +669,8 @@ static enum fsread_res ast_readaudio_callback(struct ast_filestream *s)
                        ast_settimeout(s->owner, whennext, ast_fsread_audio, s);
                else
 #endif         
-                       s->owner->streamid = ast_sched_add(s->owner->sched, whennext / 8, ast_fsread_audio, s);
+                       s->owner->streamid = ast_sched_add(s->owner->sched, 
+                               whennext / (ast_format_rate(s->fmt->format) / 1000), ast_fsread_audio, s);
                s->lasttimeout = whennext;
                return FSREAD_SUCCESS_NOSCHED;
        }
@@ -713,7 +714,8 @@ static enum fsread_res ast_readvideo_callback(struct ast_filestream *s)
        }
 
        if (whennext != s->lasttimeout) {
-               s->owner->vstreamid = ast_sched_add(s->owner->sched, whennext / 8, 
+               s->owner->vstreamid = ast_sched_add(s->owner->sched, 
+                       whennext / (ast_format_rate(s->fmt->format) / 1000), 
                        ast_fsread_video, s);
                s->lasttimeout = whennext;
                return FSREAD_SUCCESS_NOSCHED;
index 433fa2c..c449b8b 100644 (file)
@@ -3186,8 +3186,14 @@ int ast_rtp_write(struct ast_rtp *rtp, struct ast_frame *_f)
                        ast_smoother_feed(rtp->smoother, _f);
                }
 
-               while ((f = ast_smoother_read(rtp->smoother)) && (f->data))
+               while ((f = ast_smoother_read(rtp->smoother)) && (f->data)) {
+                       if (f->subclass == AST_FORMAT_G722) {
+                               /* G.722 is silllllllllllllly */
+                               f->samples /= 2;
+                       }
+
                        ast_rtp_raw_write(rtp, f, codec);
+               }
        } else {
                /* Don't buffer outgoing frames; send them one-per-packet: */
                if (_f->offset < hdrlen)