diff mbox series

[FFmpeg-devel,1/2] avcodec: move AVCodecParameters related functions from util.c to a new file

Message ID 20210317135603.3634-1-jamrial@gmail.com
State Accepted
Commit 4057a060f62b2f1dade799fb9fdf0eec5708e0a7
Headers show
Series [FFmpeg-devel,1/2] avcodec: move AVCodecParameters related functions from util.c to a new file | expand

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

James Almer March 17, 2021, 1:56 p.m. UTC
Signed-off-by: James Almer <jamrial@gmail.com>
---
 libavcodec/Makefile    |   1 +
 libavcodec/codec_par.c | 202 +++++++++++++++++++++++++++++++++++++++++
 libavcodec/utils.c     | 173 -----------------------------------
 3 files changed, 203 insertions(+), 173 deletions(-)
 create mode 100644 libavcodec/codec_par.c

Comments

James Almer March 19, 2021, 6:18 p.m. UTC | #1
On 3/17/2021 10:56 AM, James Almer wrote:
> Signed-off-by: James Almer <jamrial@gmail.com>
> ---
>   libavcodec/Makefile    |   1 +
>   libavcodec/codec_par.c | 202 +++++++++++++++++++++++++++++++++++++++++
>   libavcodec/utils.c     | 173 -----------------------------------
>   3 files changed, 203 insertions(+), 173 deletions(-)
>   create mode 100644 libavcodec/codec_par.c

Will apply set.
diff mbox series

Patch

diff --git a/libavcodec/Makefile b/libavcodec/Makefile
index 81cc16471b..baf712129f 100644
--- a/libavcodec/Makefile
+++ b/libavcodec/Makefile
@@ -37,6 +37,7 @@  OBJS = ac3_parser.o                                                     \
        bitstream_filters.o                                              \
        bsf.o                                                            \
        codec_desc.o                                                     \
+       codec_par.o                                                      \
        d3d11va.o                                                        \
        decode.o                                                         \
        dirac.o                                                          \
diff --git a/libavcodec/codec_par.c b/libavcodec/codec_par.c
new file mode 100644
index 0000000000..1a5168a04b
--- /dev/null
+++ b/libavcodec/codec_par.c
@@ -0,0 +1,202 @@ 
+/*
+ * AVCodecParameters functions for libavcodec
+ *
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * FFmpeg is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with FFmpeg; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+/**
+ * @file
+ * AVCodecParameters functions for libavcodec.
+ */
+
+#include <string.h>
+#include "libavutil/mem.h"
+#include "avcodec.h"
+#include "codec_par.h"
+
+static void codec_parameters_reset(AVCodecParameters *par)
+{
+    av_freep(&par->extradata);
+
+    memset(par, 0, sizeof(*par));
+
+    par->codec_type          = AVMEDIA_TYPE_UNKNOWN;
+    par->codec_id            = AV_CODEC_ID_NONE;
+    par->format              = -1;
+    par->field_order         = AV_FIELD_UNKNOWN;
+    par->color_range         = AVCOL_RANGE_UNSPECIFIED;
+    par->color_primaries     = AVCOL_PRI_UNSPECIFIED;
+    par->color_trc           = AVCOL_TRC_UNSPECIFIED;
+    par->color_space         = AVCOL_SPC_UNSPECIFIED;
+    par->chroma_location     = AVCHROMA_LOC_UNSPECIFIED;
+    par->sample_aspect_ratio = (AVRational){ 0, 1 };
+    par->profile             = FF_PROFILE_UNKNOWN;
+    par->level               = FF_LEVEL_UNKNOWN;
+}
+
+AVCodecParameters *avcodec_parameters_alloc(void)
+{
+    AVCodecParameters *par = av_mallocz(sizeof(*par));
+
+    if (!par)
+        return NULL;
+    codec_parameters_reset(par);
+    return par;
+}
+
+void avcodec_parameters_free(AVCodecParameters **ppar)
+{
+    AVCodecParameters *par = *ppar;
+
+    if (!par)
+        return;
+    codec_parameters_reset(par);
+
+    av_freep(ppar);
+}
+
+int avcodec_parameters_copy(AVCodecParameters *dst, const AVCodecParameters *src)
+{
+    codec_parameters_reset(dst);
+    memcpy(dst, src, sizeof(*dst));
+
+    dst->extradata      = NULL;
+    dst->extradata_size = 0;
+    if (src->extradata) {
+        dst->extradata = av_mallocz(src->extradata_size + AV_INPUT_BUFFER_PADDING_SIZE);
+        if (!dst->extradata)
+            return AVERROR(ENOMEM);
+        memcpy(dst->extradata, src->extradata, src->extradata_size);
+        dst->extradata_size = src->extradata_size;
+    }
+
+    return 0;
+}
+
+int avcodec_parameters_from_context(AVCodecParameters *par,
+                                    const AVCodecContext *codec)
+{
+    codec_parameters_reset(par);
+
+    par->codec_type = codec->codec_type;
+    par->codec_id   = codec->codec_id;
+    par->codec_tag  = codec->codec_tag;
+
+    par->bit_rate              = codec->bit_rate;
+    par->bits_per_coded_sample = codec->bits_per_coded_sample;
+    par->bits_per_raw_sample   = codec->bits_per_raw_sample;
+    par->profile               = codec->profile;
+    par->level                 = codec->level;
+
+    switch (par->codec_type) {
+    case AVMEDIA_TYPE_VIDEO:
+        par->format              = codec->pix_fmt;
+        par->width               = codec->width;
+        par->height              = codec->height;
+        par->field_order         = codec->field_order;
+        par->color_range         = codec->color_range;
+        par->color_primaries     = codec->color_primaries;
+        par->color_trc           = codec->color_trc;
+        par->color_space         = codec->colorspace;
+        par->chroma_location     = codec->chroma_sample_location;
+        par->sample_aspect_ratio = codec->sample_aspect_ratio;
+        par->video_delay         = codec->has_b_frames;
+        break;
+    case AVMEDIA_TYPE_AUDIO:
+        par->format           = codec->sample_fmt;
+        par->channel_layout   = codec->channel_layout;
+        par->channels         = codec->channels;
+        par->sample_rate      = codec->sample_rate;
+        par->block_align      = codec->block_align;
+        par->frame_size       = codec->frame_size;
+        par->initial_padding  = codec->initial_padding;
+        par->trailing_padding = codec->trailing_padding;
+        par->seek_preroll     = codec->seek_preroll;
+        break;
+    case AVMEDIA_TYPE_SUBTITLE:
+        par->width  = codec->width;
+        par->height = codec->height;
+        break;
+    }
+
+    if (codec->extradata) {
+        par->extradata = av_mallocz(codec->extradata_size + AV_INPUT_BUFFER_PADDING_SIZE);
+        if (!par->extradata)
+            return AVERROR(ENOMEM);
+        memcpy(par->extradata, codec->extradata, codec->extradata_size);
+        par->extradata_size = codec->extradata_size;
+    }
+
+    return 0;
+}
+
+int avcodec_parameters_to_context(AVCodecContext *codec,
+                                  const AVCodecParameters *par)
+{
+    codec->codec_type = par->codec_type;
+    codec->codec_id   = par->codec_id;
+    codec->codec_tag  = par->codec_tag;
+
+    codec->bit_rate              = par->bit_rate;
+    codec->bits_per_coded_sample = par->bits_per_coded_sample;
+    codec->bits_per_raw_sample   = par->bits_per_raw_sample;
+    codec->profile               = par->profile;
+    codec->level                 = par->level;
+
+    switch (par->codec_type) {
+    case AVMEDIA_TYPE_VIDEO:
+        codec->pix_fmt                = par->format;
+        codec->width                  = par->width;
+        codec->height                 = par->height;
+        codec->field_order            = par->field_order;
+        codec->color_range            = par->color_range;
+        codec->color_primaries        = par->color_primaries;
+        codec->color_trc              = par->color_trc;
+        codec->colorspace             = par->color_space;
+        codec->chroma_sample_location = par->chroma_location;
+        codec->sample_aspect_ratio    = par->sample_aspect_ratio;
+        codec->has_b_frames           = par->video_delay;
+        break;
+    case AVMEDIA_TYPE_AUDIO:
+        codec->sample_fmt       = par->format;
+        codec->channel_layout   = par->channel_layout;
+        codec->channels         = par->channels;
+        codec->sample_rate      = par->sample_rate;
+        codec->block_align      = par->block_align;
+        codec->frame_size       = par->frame_size;
+        codec->delay            =
+        codec->initial_padding  = par->initial_padding;
+        codec->trailing_padding = par->trailing_padding;
+        codec->seek_preroll     = par->seek_preroll;
+        break;
+    case AVMEDIA_TYPE_SUBTITLE:
+        codec->width  = par->width;
+        codec->height = par->height;
+        break;
+    }
+
+    if (par->extradata) {
+        av_freep(&codec->extradata);
+        codec->extradata = av_mallocz(par->extradata_size + AV_INPUT_BUFFER_PADDING_SIZE);
+        if (!codec->extradata)
+            return AVERROR(ENOMEM);
+        memcpy(codec->extradata, par->extradata, par->extradata_size);
+        codec->extradata_size = par->extradata_size;
+    }
+
+    return 0;
+}
diff --git a/libavcodec/utils.c b/libavcodec/utils.c
index 71dbcb19d8..33626a7106 100644
--- a/libavcodec/utils.c
+++ b/libavcodec/utils.c
@@ -1877,179 +1877,6 @@  AVCPBProperties *ff_add_cpb_side_data(AVCodecContext *avctx)
     return props;
 }
 
-static void codec_parameters_reset(AVCodecParameters *par)
-{
-    av_freep(&par->extradata);
-
-    memset(par, 0, sizeof(*par));
-
-    par->codec_type          = AVMEDIA_TYPE_UNKNOWN;
-    par->codec_id            = AV_CODEC_ID_NONE;
-    par->format              = -1;
-    par->field_order         = AV_FIELD_UNKNOWN;
-    par->color_range         = AVCOL_RANGE_UNSPECIFIED;
-    par->color_primaries     = AVCOL_PRI_UNSPECIFIED;
-    par->color_trc           = AVCOL_TRC_UNSPECIFIED;
-    par->color_space         = AVCOL_SPC_UNSPECIFIED;
-    par->chroma_location     = AVCHROMA_LOC_UNSPECIFIED;
-    par->sample_aspect_ratio = (AVRational){ 0, 1 };
-    par->profile             = FF_PROFILE_UNKNOWN;
-    par->level               = FF_LEVEL_UNKNOWN;
-}
-
-AVCodecParameters *avcodec_parameters_alloc(void)
-{
-    AVCodecParameters *par = av_mallocz(sizeof(*par));
-
-    if (!par)
-        return NULL;
-    codec_parameters_reset(par);
-    return par;
-}
-
-void avcodec_parameters_free(AVCodecParameters **ppar)
-{
-    AVCodecParameters *par = *ppar;
-
-    if (!par)
-        return;
-    codec_parameters_reset(par);
-
-    av_freep(ppar);
-}
-
-int avcodec_parameters_copy(AVCodecParameters *dst, const AVCodecParameters *src)
-{
-    codec_parameters_reset(dst);
-    memcpy(dst, src, sizeof(*dst));
-
-    dst->extradata      = NULL;
-    dst->extradata_size = 0;
-    if (src->extradata) {
-        dst->extradata = av_mallocz(src->extradata_size + AV_INPUT_BUFFER_PADDING_SIZE);
-        if (!dst->extradata)
-            return AVERROR(ENOMEM);
-        memcpy(dst->extradata, src->extradata, src->extradata_size);
-        dst->extradata_size = src->extradata_size;
-    }
-
-    return 0;
-}
-
-int avcodec_parameters_from_context(AVCodecParameters *par,
-                                    const AVCodecContext *codec)
-{
-    codec_parameters_reset(par);
-
-    par->codec_type = codec->codec_type;
-    par->codec_id   = codec->codec_id;
-    par->codec_tag  = codec->codec_tag;
-
-    par->bit_rate              = codec->bit_rate;
-    par->bits_per_coded_sample = codec->bits_per_coded_sample;
-    par->bits_per_raw_sample   = codec->bits_per_raw_sample;
-    par->profile               = codec->profile;
-    par->level                 = codec->level;
-
-    switch (par->codec_type) {
-    case AVMEDIA_TYPE_VIDEO:
-        par->format              = codec->pix_fmt;
-        par->width               = codec->width;
-        par->height              = codec->height;
-        par->field_order         = codec->field_order;
-        par->color_range         = codec->color_range;
-        par->color_primaries     = codec->color_primaries;
-        par->color_trc           = codec->color_trc;
-        par->color_space         = codec->colorspace;
-        par->chroma_location     = codec->chroma_sample_location;
-        par->sample_aspect_ratio = codec->sample_aspect_ratio;
-        par->video_delay         = codec->has_b_frames;
-        break;
-    case AVMEDIA_TYPE_AUDIO:
-        par->format           = codec->sample_fmt;
-        par->channel_layout   = codec->channel_layout;
-        par->channels         = codec->channels;
-        par->sample_rate      = codec->sample_rate;
-        par->block_align      = codec->block_align;
-        par->frame_size       = codec->frame_size;
-        par->initial_padding  = codec->initial_padding;
-        par->trailing_padding = codec->trailing_padding;
-        par->seek_preroll     = codec->seek_preroll;
-        break;
-    case AVMEDIA_TYPE_SUBTITLE:
-        par->width  = codec->width;
-        par->height = codec->height;
-        break;
-    }
-
-    if (codec->extradata) {
-        par->extradata = av_mallocz(codec->extradata_size + AV_INPUT_BUFFER_PADDING_SIZE);
-        if (!par->extradata)
-            return AVERROR(ENOMEM);
-        memcpy(par->extradata, codec->extradata, codec->extradata_size);
-        par->extradata_size = codec->extradata_size;
-    }
-
-    return 0;
-}
-
-int avcodec_parameters_to_context(AVCodecContext *codec,
-                                  const AVCodecParameters *par)
-{
-    codec->codec_type = par->codec_type;
-    codec->codec_id   = par->codec_id;
-    codec->codec_tag  = par->codec_tag;
-
-    codec->bit_rate              = par->bit_rate;
-    codec->bits_per_coded_sample = par->bits_per_coded_sample;
-    codec->bits_per_raw_sample   = par->bits_per_raw_sample;
-    codec->profile               = par->profile;
-    codec->level                 = par->level;
-
-    switch (par->codec_type) {
-    case AVMEDIA_TYPE_VIDEO:
-        codec->pix_fmt                = par->format;
-        codec->width                  = par->width;
-        codec->height                 = par->height;
-        codec->field_order            = par->field_order;
-        codec->color_range            = par->color_range;
-        codec->color_primaries        = par->color_primaries;
-        codec->color_trc              = par->color_trc;
-        codec->colorspace             = par->color_space;
-        codec->chroma_sample_location = par->chroma_location;
-        codec->sample_aspect_ratio    = par->sample_aspect_ratio;
-        codec->has_b_frames           = par->video_delay;
-        break;
-    case AVMEDIA_TYPE_AUDIO:
-        codec->sample_fmt       = par->format;
-        codec->channel_layout   = par->channel_layout;
-        codec->channels         = par->channels;
-        codec->sample_rate      = par->sample_rate;
-        codec->block_align      = par->block_align;
-        codec->frame_size       = par->frame_size;
-        codec->delay            =
-        codec->initial_padding  = par->initial_padding;
-        codec->trailing_padding = par->trailing_padding;
-        codec->seek_preroll     = par->seek_preroll;
-        break;
-    case AVMEDIA_TYPE_SUBTITLE:
-        codec->width  = par->width;
-        codec->height = par->height;
-        break;
-    }
-
-    if (par->extradata) {
-        av_freep(&codec->extradata);
-        codec->extradata = av_mallocz(par->extradata_size + AV_INPUT_BUFFER_PADDING_SIZE);
-        if (!codec->extradata)
-            return AVERROR(ENOMEM);
-        memcpy(codec->extradata, par->extradata, par->extradata_size);
-        codec->extradata_size = par->extradata_size;
-    }
-
-    return 0;
-}
-
 static unsigned bcd2uint(uint8_t bcd)
 {
     unsigned low  = bcd & 0xf;