[FFmpeg-devel] [PATCH 2/5] cbs: Add a macro to create functions for deep copying
Andreas Rheinhardt
andreas.rheinhardt at googlemail.com
Mon Nov 26 15:39:10 EET 2018
Some structs in cbs (most importantly some of the parameter sets of
H.264/HEVC) can contain external buffers so that shallow copies are not
enough for them; furthermore, they need a special free-callback
function. The macro provided in this commit can be used to easily create
functions both for copying as well as freeing such structures.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at googlemail.com>
---
libavcodec/cbs_internal.h | 53 +++++++++++++++++++++++++++++++++++++++
1 file changed, 53 insertions(+)
diff --git a/libavcodec/cbs_internal.h b/libavcodec/cbs_internal.h
index 62a836af90..dd45748c61 100644
--- a/libavcodec/cbs_internal.h
+++ b/libavcodec/cbs_internal.h
@@ -98,4 +98,57 @@ extern const CodedBitstreamType ff_cbs_type_mpeg2;
extern const CodedBitstreamType ff_cbs_type_vp9;
+enum {
+ BITS,
+ BYTES
+};
+
+// The following macro automatically creates both (deep) copy and
+// free functions for structs with exactly one internal buffer.
+
+#define cbs_copy_free(codec, type, var, buffer, size_element, size_offset, size_unit) \
+static void cbs_ ## codec ## _free_ ## var(void *unit, uint8_t *content) \
+{ \
+ type *var = (type *)content; \
+ \
+ av_buffer_unref(&var->buffer ## _ref); \
+ av_freep(&var); \
+} \
+ \
+static AVBufferRef *cbs_ ## codec ## _copy_ ## var(const type *source) \
+{ \
+ AVBufferRef *copy_ref; \
+ type *copy; \
+ \
+ copy = av_malloc(sizeof(type)); \
+ if (!copy) \
+ return NULL; \
+ memcpy(copy, source, sizeof(type)); \
+ \
+ copy_ref = av_buffer_create((uint8_t*)copy, sizeof(type), \
+ &cbs_ ## codec ## _free_ ## var, \
+ NULL, 0); \
+ if (!copy_ref) { \
+ av_free(copy); \
+ return NULL; \
+ } \
+ \
+ if (source->buffer) { \
+ size_t size = (size_t)source->size_element + size_offset; \
+ if (size_unit == BITS) \
+ size = (size + 7) / 8; \
+ \
+ copy->buffer ## _ref = av_buffer_alloc(size + AV_INPUT_BUFFER_PADDING_SIZE); \
+ if (!copy->buffer ## _ref) {\
+ av_buffer_unref(©_ref); \
+ return NULL; \
+ } \
+ copy->buffer = copy->buffer ## _ref->data; \
+ memcpy(copy->buffer, source->buffer, size); \
+ memset(copy->buffer + size, 0, AV_INPUT_BUFFER_PADDING_SIZE); \
+ } \
+ \
+ return copy_ref; \
+}
+
#endif /* AVCODEC_CBS_INTERNAL_H */
--
2.19.1
More information about the ffmpeg-devel
mailing list