Remove unnecessary assignment, found by CSA.

[ffmpeg] / libavcodec / libschroedingerdec.c

diff --git a/libavcodec/libschroedingerdec.c b/libavcodec/libschroedingerdec.c
index 2f8c88f59c4f12492ab84b3c4fb481389942fd69..3985a1ca2de2e1ad7f824a66377454f73d2d9ff2 100644 (file)
--- a/libavcodec/libschroedingerdec.c
+++ b/libavcodec/libschroedingerdec.c
@@ -20,7 +20,7 @@
  */
 
 /**
-* @file libschroedingerdec.c
+* @file libavcodec/libschroedingerdec.c
 * Dirac decoder support via libschroedinger-1.0 libraries. More details about
 * the Schroedinger project can be found at http://www.diracvideo.org/.
 * The library implements Dirac Specification Version 2.2.
@@ -64,6 +64,60 @@ typedef struct FfmpegSchroDecoderParams
     AVPicture dec_pic;
 } FfmpegSchroDecoderParams;
 
+typedef struct FfmpegSchroParseUnitContext
+{
+    const uint8_t *buf;
+    int           buf_size;
+} FfmpegSchroParseUnitContext;
+
+
+static void libschroedinger_decode_buffer_free (SchroBuffer *schro_buf,
+                                                void *priv);
+
+static void FfmpegSchroParseContextInit (FfmpegSchroParseUnitContext *parse_ctx,
+                                         const uint8_t *buf, int buf_size)
+{
+    parse_ctx->buf           = buf;
+    parse_ctx->buf_size      = buf_size;
+}
+
+static SchroBuffer* FfmpegFindNextSchroParseUnit (FfmpegSchroParseUnitContext *parse_ctx)
+{
+    SchroBuffer *enc_buf = NULL;
+    int next_pu_offset = 0;
+    unsigned char *in_buf;
+
+    if (parse_ctx->buf_size < 13 ||
+        parse_ctx->buf[0] != 'B' ||
+        parse_ctx->buf[1] != 'B' ||
+        parse_ctx->buf[2] != 'C' ||
+        parse_ctx->buf[3] != 'D')
+        return NULL;
+
+    next_pu_offset = (parse_ctx->buf[5] << 24) +
+                     (parse_ctx->buf[6] << 16) +
+                     (parse_ctx->buf[7] <<  8) +
+                      parse_ctx->buf[8];
+
+    if (next_pu_offset == 0 &&
+        SCHRO_PARSE_CODE_IS_END_OF_SEQUENCE(parse_ctx->buf[4]))
+        next_pu_offset = 13;
+
+    if (next_pu_offset <= 0 || parse_ctx->buf_size < next_pu_offset)
+        return NULL;
+
+    in_buf = av_malloc(next_pu_offset);
+    memcpy (in_buf, parse_ctx->buf, next_pu_offset);
+    enc_buf = schro_buffer_new_with_data (in_buf, next_pu_offset);
+    enc_buf->free = libschroedinger_decode_buffer_free;
+    enc_buf->priv = in_buf;
+
+    parse_ctx->buf += next_pu_offset;
+    parse_ctx->buf_size -= next_pu_offset;
+
+    return enc_buf;
+}
+
 /**
 * Returns FFmpeg chroma format.
 */
@@ -81,9 +135,7 @@ static enum PixelFormat GetFfmpegChromaFormat(SchroChromaFormat schro_pix_fmt)
     return PIX_FMT_NONE;
 }
 
-/*-------------------------DECODER------------------------------------------*/
-
-static int libschroedinger_decode_init(AVCodecContext *avccontext)
+static av_cold int libschroedinger_decode_init(AVCodecContext *avccontext)
 {
 
     FfmpegSchroDecoderParams *p_schro_params = avccontext->priv_data ;
@@ -126,7 +178,7 @@ static void libschroedinger_handle_first_access_unit(AVCodecContext *avccontext)
         av_log(avccontext, AV_LOG_ERROR, "invalid dimensions (%dx%d)\n",
                p_schro_params->format->width, p_schro_params->format->height);
         avccontext->height = avccontext->width = 0;
-        return -1;
+        return;
     }
     avccontext->height  = p_schro_params->format->height;
     avccontext->width   = p_schro_params->format->width;
@@ -138,7 +190,7 @@ static void libschroedinger_handle_first_access_unit(AVCodecContext *avccontext)
         av_log (avccontext, AV_LOG_ERROR,
                 "This codec currently only supports planar YUV 4:2:0, 4:2:2 "
                 "and 4:4:4 formats.\n");
-        return -1;
+        return;
     }
 
     avccontext->time_base.den = p_schro_params->format->frame_rate_numerator;
@@ -155,8 +207,10 @@ static void libschroedinger_handle_first_access_unit(AVCodecContext *avccontext)
 
 static int libschroedinger_decode_frame(AVCodecContext *avccontext,
                                         void *data, int *data_size,
-                                        const uint8_t *buf, int buf_size)
+                                        AVPacket *avpkt)
 {
+    const uint8_t *buf = avpkt->data;
+    int buf_size = avpkt->size;
 
     FfmpegSchroDecoderParams *p_schro_params = avccontext->priv_data;
     SchroDecoder *decoder = p_schro_params->decoder;
@@ -166,26 +220,33 @@ static int libschroedinger_decode_frame(AVCodecContext *avccontext,
     SchroFrame* frame;
     int state;
     int go = 1;
+    int outer = 1;
+    FfmpegSchroParseUnitContext parse_ctx;
 
     *data_size = 0;
 
-    if (buf_size>0) {
-        unsigned char *in_buf = av_malloc(buf_size);
-        memcpy (in_buf, buf, buf_size);
-        enc_buf = schro_buffer_new_with_data (in_buf, buf_size);
-        enc_buf->free = libschroedinger_decode_buffer_free;
-        enc_buf->priv = in_buf;
-        /* Push buffer into decoder. */
-        state = schro_decoder_push (decoder, enc_buf);
-        if (state == SCHRO_DECODER_FIRST_ACCESS_UNIT)
-            libschroedinger_handle_first_access_unit(avccontext);
-    } else {
+    FfmpegSchroParseContextInit (&parse_ctx, buf, buf_size);
+    if (buf_size == 0) {
         if (!p_schro_params->eos_signalled) {
             state = schro_decoder_push_end_of_stream(decoder);
             p_schro_params->eos_signalled = 1;
         }
     }
 
+    /* Loop through all the individual parse units in the input buffer */
+    do {
+        if ((enc_buf = FfmpegFindNextSchroParseUnit(&parse_ctx))) {
+            /* Push buffer into decoder. */
+            if (SCHRO_PARSE_CODE_IS_PICTURE(enc_buf->data[4]) &&
+                SCHRO_PARSE_CODE_NUM_REFS(enc_buf->data[4]) > 0)
+                avccontext->has_b_frames = 1;
+            state = schro_decoder_push (decoder, enc_buf);
+            if (state == SCHRO_DECODER_FIRST_ACCESS_UNIT)
+                  libschroedinger_handle_first_access_unit(avccontext);
+            go = 1;
+        }
+        else
+            outer = 0;
     format = p_schro_params->format;
 
     while (go) {
@@ -226,6 +287,7 @@ static int libschroedinger_decode_frame(AVCodecContext *avccontext,
             go = 0;
             p_schro_params->eos_pulled = 1;
             schro_decoder_reset (decoder);
+            outer = 0;
             break;
 
         case SCHRO_DECODER_ERROR:
@@ -233,6 +295,7 @@ static int libschroedinger_decode_frame(AVCodecContext *avccontext,
             break;
         }
     }
+    } while(outer);
 
     /* Grab next frame to be returned from the top of the queue. */
     frame = ff_dirac_schro_queue_pop(&p_schro_params->dec_frame_queue);
@@ -264,7 +327,7 @@ static int libschroedinger_decode_frame(AVCodecContext *avccontext,
 }
 
 
-static int libschroedinger_decode_close(AVCodecContext *avccontext)
+static av_cold int libschroedinger_decode_close(AVCodecContext *avccontext)
 {
     FfmpegSchroDecoderParams *p_schro_params = avccontext->priv_data;
     /* Free the decoder. */
@@ -306,5 +369,6 @@ AVCodec libschroedinger_decoder = {
     libschroedinger_decode_close,
     libschroedinger_decode_frame,
     CODEC_CAP_DELAY,
-    .flush = libschroedinger_flush
+    .flush = libschroedinger_flush,
+    .long_name = NULL_IF_CONFIG_SMALL("libschroedinger Dirac 2.2"),
 };