opusenc: Add some FIXME comments about calculating padding with LPC
authorSebastian Dröge <sebastian@centricular.com>
Sat, 31 Oct 2015 13:02:50 +0000 (15:02 +0200)
committerSebastian Dröge <sebastian@centricular.com>
Tue, 3 Nov 2015 18:35:41 +0000 (20:35 +0200)
https://bugzilla.gnome.org/show_bug.cgi?id=757153

ext/opus/gstopusenc.c

index 94c43dd..718643a 100644 (file)
@@ -960,6 +960,10 @@ gst_opus_enc_encode (GstOpusEnc * enc, GstBuffer * buf)
 
       size = ((bsize / bytes) + 1) * bytes;
       mdata = g_malloc0 (size);
+      /* FIXME: Instead of silence, use LPC with the last real samples.
+       * Otherwise we will create a discontinuity here, which will distort the
+       * last few encoded samples
+       */
       memcpy (mdata, bdata, bsize);
       data = mdata;
     } else {
@@ -981,6 +985,10 @@ gst_opus_enc_encode (GstOpusEnc * enc, GstBuffer * buf)
     }
   } else {
     if (enc->encoded_samples < enc->consumed_samples) {
+      /* FIXME: Instead of silence, use LPC with the last real samples.
+       * Otherwise we will create a discontinuity here, which will distort the
+       * last few encoded samples
+       */
       data = mdata = g_malloc0 (bytes);
       size = bytes;
       output_samples = enc->consumed_samples - enc->encoded_samples;