[Bug 796519] Add AV1 codec parser

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Tue Aug 14 14:53:35 UTC 2018


https://bugzilla.gnome.org/show_bug.cgi?id=796519

Víctor Manuel Jáquez Leal <vjaquez at igalia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #373286|none                        |reviewed
             status|                            |

--- Comment #42 from Víctor Manuel Jáquez Leal <vjaquez at igalia.com> ---
Review of attachment 373286:
 --> (https://bugzilla.gnome.org/review?bug=796519&attachment=373286)

as this parser needs to allocate memory and free it, perhaps it is required to
add the glib's annotation for autoclean pointers.

::: gst-libs/gst/codecparsers/gstav1parser.c
@@ +167,3 @@
+#define gst_av1_bit_reader_skip(br, nbits) if
(!gst_bit_reader_skip(br,(nbits))) { \
+                                             GST_LOG("Skip Bit Error at line
%d",__LINE__); \
+                                             return
GST_AV1_PARSER_SKIPBITS_ERROR; \

inlined function instead?

@@ +186,3 @@
+
+static gint
+gst_av1_helpers_FloorLog2 (guint32 x)

As this is an inside helper, I would keep the low_case format (not CamelCase),
without the namespace
for example,

_floor_log2(..

@@ +198,3 @@
+
+static gint
+gst_av1_helper_TileLog2 (gint blkSize, gint target)

ditto

_tile_log2(..

@@ +207,3 @@
+
+static gint
+gst_av1_helper_InverseRecenter (gint r, gint v)

ditto

_inverse_recenter(..

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list