Fix compilation without HAVE_AVX, HAVE_YASM etc.

[ffmpeg] / libavcodec / libcelt_dec.c

diff --git a/libavcodec/libcelt_dec.c b/libavcodec/libcelt_dec.c
index 6f3965401cac9e519047d5ce661e1d7536f393ae..ffc608f2347b9163a7e45d245d68383585f2d362 100644 (file)
--- a/libavcodec/libcelt_dec.c
+++ b/libavcodec/libcelt_dec.c
@@ -1,5 +1,5 @@
 /*
- * Xiph CELT / Opus decoder using libcelt
+ * Xiph CELT decoder using libcelt
  * Copyright (c) 2011 Nicolas George
  *
  * This file is part of FFmpeg.
@@ -27,20 +27,20 @@
 struct libcelt_context {
     CELTMode *mode;
     CELTDecoder *dec;
-    int frame_bytes;
+    AVFrame frame;
     int discard;
 };
 
 static int ff_celt_error_to_averror(int err)
 {
-    switch(err) {
+    switch (err) {
         case CELT_BAD_ARG:          return AVERROR(EINVAL);
 #ifdef CELT_BUFFER_TOO_SMALL
         case CELT_BUFFER_TOO_SMALL: return AVERROR(ENOBUFS);
 #endif
         case CELT_INTERNAL_ERROR:   return AVERROR(EFAULT);
         case CELT_CORRUPTED_DATA:   return AVERROR_INVALIDDATA;
-        case CELT_UNIMPLEMENTED:    return AVERROR(ENOTSUP);
+        case CELT_UNIMPLEMENTED:    return AVERROR(ENOSYS);
 #ifdef ENOTRECOVERABLE
         case CELT_INVALID_STATE:    return AVERROR(ENOTRECOVERABLE);
 #endif
@@ -64,7 +64,6 @@ static av_cold int libcelt_dec_init(AVCodecContext *c)
     if (!c->channels || !c->frame_size ||
         c->frame_size > INT_MAX / sizeof(int16_t) / c->channels)
         return AVERROR(EINVAL);
-    celt->frame_bytes = c->frame_size * c->channels * sizeof(int16_t);
     celt->mode = celt_mode_create(c->sample_rate, c->frame_size, &err);
     if (!celt->mode)
         return ff_celt_error_to_averror(err);
@@ -80,9 +79,8 @@ static av_cold int libcelt_dec_init(AVCodecContext *c)
                    "Invalid overlap (%d), ignored.\n", celt->discard);
             celt->discard = 0;
         }
-        celt->discard *= c->channels * sizeof(int16_t);
     }
-    if(c->extradata_size >= 8) {
+    if (c->extradata_size >= 8) {
         unsigned version = AV_RL32(c->extradata + 4);
         unsigned lib_version = ff_celt_bitstream_version_hack(celt->mode);
         if (version != lib_version)
@@ -91,6 +89,9 @@ static av_cold int libcelt_dec_init(AVCodecContext *c)
                    "improperly decoded by libcelt for version 0x%x.\n",
                    version, lib_version);
     }
+    c->sample_fmt = AV_SAMPLE_FMT_S16;
+    avcodec_get_frame_defaults(&celt->frame);
+    c->coded_frame = &celt->frame;
     return 0;
 }
 
@@ -103,23 +104,31 @@ static av_cold int libcelt_dec_close(AVCodecContext *c)
     return 0;
 }
 
-static int libcelt_dec_decode(AVCodecContext *c, void *pcm, int *pcm_size,
-                              AVPacket *pkt)
+static int libcelt_dec_decode(AVCodecContext *c, void *frame,
+                              int *got_frame_ptr, AVPacket *pkt)
 {
     struct libcelt_context *celt = c->priv_data;
     int err;
+    int16_t *pcm;
 
-    if (*pcm_size < celt->frame_bytes)
-        return AVERROR(ENOBUFS);
+    celt->frame.nb_samples = c->frame_size;
+    err = c->get_buffer(c, &celt->frame);
+    if (err < 0) {
+        av_log(c, AV_LOG_ERROR, "get_buffer() failed\n");
+        return err;
+    }
+    pcm = (int16_t *)celt->frame.data[0];
     err = celt_decode(celt->dec, pkt->data, pkt->size, pcm, c->frame_size);
     if (err < 0)
         return ff_celt_error_to_averror(err);
-    *pcm_size = celt->frame_bytes;
     if (celt->discard) {
-        *pcm_size = celt->frame_bytes - celt->discard;
-        memmove(pcm, (char *)pcm + celt->discard, *pcm_size);
+        celt->frame.nb_samples -= celt->discard;
+        memmove(pcm, pcm + celt->discard * c->channels,
+                celt->frame.nb_samples * c->channels * sizeof(int16_t));
         celt->discard = 0;
     }
+    *got_frame_ptr = 1;
+    *(AVFrame *)frame = celt->frame;
     return pkt->size;
 }
 
@@ -131,6 +140,6 @@ AVCodec ff_libcelt_decoder = {
     .init           = libcelt_dec_init,
     .close          = libcelt_dec_close,
     .decode         = libcelt_dec_decode,
-    .capabilities   = 0,
-    .long_name = NULL_IF_CONFIG_SMALL("Xiph CELT/Opus decoder using libcelt"),
+    .capabilities   = CODEC_CAP_DR1,
+    .long_name      = NULL_IF_CONFIG_SMALL("Xiph CELT decoder using libcelt"),
 };