video: Add/fix various annotations
authorSebastian Dröge <sebastian@centricular.com>
Fri, 14 Oct 2022 18:59:43 +0000 (21:59 +0300)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Tue, 18 Oct 2022 08:56:58 +0000 (08:56 +0000)
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3194>

12 files changed:
subprojects/gst-plugins-base/gst-libs/gst/video/gstvideodecoder.c
subprojects/gst-plugins-base/gst-libs/gst/video/gstvideoencoder.c
subprojects/gst-plugins-base/gst-libs/gst/video/gstvideometa.c
subprojects/gst-plugins-base/gst-libs/gst/video/gstvideotimecode.c
subprojects/gst-plugins-base/gst-libs/gst/video/video-anc.c
subprojects/gst-plugins-base/gst-libs/gst/video/video-anc.h
subprojects/gst-plugins-base/gst-libs/gst/video/video-converter.c
subprojects/gst-plugins-base/gst-libs/gst/video/video-format.c
subprojects/gst-plugins-base/gst-libs/gst/video/video-info.c
subprojects/gst-plugins-base/gst-libs/gst/video/video-multiview.c
subprojects/gst-plugins-base/gst-libs/gst/video/video-overlay-composition.c
subprojects/gst-plugins-base/gst-libs/gst/video/video-sei.h

index 43b5e52..fd3eeb1 100644 (file)
@@ -2036,8 +2036,8 @@ gst_video_decoder_src_query (GstPad * pad, GstObject * parent, GstQuery * query)
 /**
  * gst_video_decoder_proxy_getcaps:
  * @decoder: a #GstVideoDecoder
- * @caps: (allow-none): initial caps
- * @filter: (allow-none): filter caps
+ * @caps: (nullable): initial caps
+ * @filter: (nullable): filter caps
  *
  * Returns caps that express @caps (or sink template caps if @caps == NULL)
  * restricted to resolution/format/... combinations supported by downstream
@@ -4006,7 +4006,7 @@ gst_video_decoder_decode_frame (GstVideoDecoder * decoder,
  *
  * Get the #GstVideoCodecState currently describing the output stream.
  *
- * Returns: (transfer full): #GstVideoCodecState describing format of video data.
+ * Returns: (transfer full) (nullable): #GstVideoCodecState describing format of video data.
  */
 GstVideoCodecState *
 gst_video_decoder_get_output_state (GstVideoDecoder * decoder)
@@ -4074,7 +4074,7 @@ _set_interlaced_output_state (GstVideoDecoder * decoder,
  * @fmt: a #GstVideoFormat
  * @width: The width in pixels
  * @height: The height in pixels
- * @reference: (allow-none) (transfer none): An optional reference #GstVideoCodecState
+ * @reference: (nullable) (transfer none): An optional reference #GstVideoCodecState
  *
  * Creates a new #GstVideoCodecState with the specified @fmt, @width and @height
  * as the output state for the decoder.
@@ -4091,7 +4091,7 @@ _set_interlaced_output_state (GstVideoDecoder * decoder,
  * The new output state will only take effect (set on pads and buffers) starting
  * from the next call to #gst_video_decoder_finish_frame().
  *
- * Returns: (transfer full): the newly configured output state.
+ * Returns: (transfer full) (nullable): the newly configured output state.
  */
 GstVideoCodecState *
 gst_video_decoder_set_output_state (GstVideoDecoder * decoder,
@@ -4109,12 +4109,12 @@ gst_video_decoder_set_output_state (GstVideoDecoder * decoder,
  * @width: The width in pixels
  * @height: The height in pixels
  * @interlace_mode: A #GstVideoInterlaceMode
- * @reference: (allow-none) (transfer none): An optional reference #GstVideoCodecState
+ * @reference: (nullable) (transfer none): An optional reference #GstVideoCodecState
  *
  * Same as #gst_video_decoder_set_output_state() but also allows you to also set
  * the interlacing mode.
  *
- * Returns: (transfer full): the newly configured output state.
+ * Returns: (transfer full) (nullable): the newly configured output state.
  *
  * Since: 1.16.
  */
@@ -4134,7 +4134,7 @@ gst_video_decoder_set_interlaced_output_state (GstVideoDecoder * decoder,
  *
  * Get the oldest pending unfinished #GstVideoCodecFrame
  *
- * Returns: (transfer full): oldest pending unfinished #GstVideoCodecFrame.
+ * Returns: (transfer full) (nullable): oldest pending unfinished #GstVideoCodecFrame.
  */
 GstVideoCodecFrame *
 gst_video_decoder_get_oldest_frame (GstVideoDecoder * decoder)
@@ -4156,7 +4156,7 @@ gst_video_decoder_get_oldest_frame (GstVideoDecoder * decoder)
  *
  * Get a pending unfinished #GstVideoCodecFrame
  *
- * Returns: (transfer full): pending unfinished #GstVideoCodecFrame identified by @frame_number.
+ * Returns: (transfer full) (nullable): pending unfinished #GstVideoCodecFrame identified by @frame_number.
  */
 GstVideoCodecFrame *
 gst_video_decoder_get_frame (GstVideoDecoder * decoder, int frame_number)
@@ -4565,7 +4565,7 @@ gst_video_decoder_negotiate (GstVideoDecoder * decoder)
  * You should use gst_video_decoder_allocate_output_frame() instead of this
  * function, if possible at all.
  *
- * Returns: (transfer full): allocated buffer, or NULL if no buffer could be
+ * Returns: (transfer full) (nullable): allocated buffer, or NULL if no buffer could be
  *     allocated (e.g. when downstream is flushing or shutting down)
  */
 GstBuffer *
@@ -5084,9 +5084,9 @@ gst_video_decoder_set_latency (GstVideoDecoder * decoder,
 /**
  * gst_video_decoder_get_latency:
  * @decoder: a #GstVideoDecoder
- * @min_latency: (out) (allow-none): address of variable in which to store the
+ * @min_latency: (out) (optional): address of variable in which to store the
  *     configured minimum latency, or %NULL
- * @max_latency: (out) (allow-none): address of variable in which to store the
+ * @max_latency: (out) (optional): address of variable in which to store the
  *     configured mximum latency, or %NULL
  *
  * Query the configured decoder latency. Results will be returned via
@@ -5107,7 +5107,7 @@ gst_video_decoder_get_latency (GstVideoDecoder * decoder,
 /**
  * gst_video_decoder_merge_tags:
  * @decoder: a #GstVideoDecoder
- * @tags: (allow-none): a #GstTagList to merge, or NULL to unset
+ * @tags: (nullable): a #GstTagList to merge, or NULL to unset
  *     previously-set tags
  * @mode: the #GstTagMergeMode to use, usually #GST_TAG_MERGE_REPLACE
  *
@@ -5150,7 +5150,7 @@ gst_video_decoder_merge_tags (GstVideoDecoder * decoder,
  * gst_video_decoder_get_buffer_pool:
  * @decoder: a #GstVideoDecoder
  *
- * Returns: (transfer full): the instance of the #GstBufferPool used
+ * Returns: (transfer full) (nullable): the instance of the #GstBufferPool used
  * by the decoder; free it after use it
  */
 GstBufferPool *
@@ -5167,9 +5167,9 @@ gst_video_decoder_get_buffer_pool (GstVideoDecoder * decoder)
 /**
  * gst_video_decoder_get_allocator:
  * @decoder: a #GstVideoDecoder
- * @allocator: (out) (allow-none) (transfer full): the #GstAllocator
+ * @allocator: (out) (optional) (nullable) (transfer full): the #GstAllocator
  * used
- * @params: (out) (allow-none) (transfer full): the
+ * @params: (out) (optional) (transfer full): the
  * #GstAllocationParams of @allocator
  *
  * Lets #GstVideoDecoder sub-classes to know the memory @allocator
index fb86cf0..2f814ad 100644 (file)
@@ -786,8 +786,8 @@ parse_fail:
 /**
  * gst_video_encoder_proxy_getcaps:
  * @enc: a #GstVideoEncoder
- * @caps: (allow-none): initial caps
- * @filter: (allow-none): filter caps
+ * @caps: (nullable): initial caps
+ * @filter: (nullable): filter caps
  *
  * Returns caps that express @caps (or sink template caps if @caps == NULL)
  * restricted to resolution/format/... combinations supported by downstream
@@ -2773,15 +2773,16 @@ done:
  *
  * Get the current #GstVideoCodecState
  *
- * Returns: (transfer full): #GstVideoCodecState describing format of video data.
+ * Returns: (transfer full) (nullable): #GstVideoCodecState describing format of video data.
  */
 GstVideoCodecState *
 gst_video_encoder_get_output_state (GstVideoEncoder * encoder)
 {
-  GstVideoCodecState *state;
+  GstVideoCodecState *state = NULL;
 
   GST_VIDEO_ENCODER_STREAM_LOCK (encoder);
-  state = gst_video_codec_state_ref (encoder->priv->output_state);
+  if (encoder->priv->output_state)
+    state = gst_video_codec_state_ref (encoder->priv->output_state);
   GST_VIDEO_ENCODER_STREAM_UNLOCK (encoder);
 
   return state;
@@ -2791,7 +2792,7 @@ gst_video_encoder_get_output_state (GstVideoEncoder * encoder)
  * gst_video_encoder_set_output_state:
  * @encoder: a #GstVideoEncoder
  * @caps: (transfer full): the #GstCaps to use for the output
- * @reference: (allow-none) (transfer none): An optional reference @GstVideoCodecState
+ * @reference: (nullable) (transfer none): An optional reference @GstVideoCodecState
  *
  * Creates a new #GstVideoCodecState with the specified caps as the output state
  * for the encoder.
@@ -2812,7 +2813,7 @@ gst_video_encoder_get_output_state (GstVideoEncoder * encoder)
  * The new output state will only take effect (set on pads and buffers) starting
  * from the next call to #gst_video_encoder_finish_frame().
  *
- * Returns: (transfer full): the newly configured output state.
+ * Returns: (transfer full) (nullable): the newly configured output state.
  */
 GstVideoCodecState *
 gst_video_encoder_set_output_state (GstVideoEncoder * encoder, GstCaps * caps,
@@ -2873,9 +2874,9 @@ gst_video_encoder_set_latency (GstVideoEncoder * encoder,
 /**
  * gst_video_encoder_get_latency:
  * @encoder: a #GstVideoEncoder
- * @min_latency: (out) (allow-none): address of variable in which to store the
+ * @min_latency: (out) (optional): address of variable in which to store the
  *     configured minimum latency, or %NULL
- * @max_latency: (out) (allow-none): address of variable in which to store the
+ * @max_latency: (out) (optional): address of variable in which to store the
  *     configured maximum latency, or %NULL
  *
  * Query the configured encoding latency. Results will be returned via
@@ -2899,7 +2900,7 @@ gst_video_encoder_get_latency (GstVideoEncoder * encoder,
  *
  * Get the oldest unfinished pending #GstVideoCodecFrame
  *
- * Returns: (transfer full): oldest unfinished pending #GstVideoCodecFrame
+ * Returns: (transfer full) (nullable): oldest unfinished pending #GstVideoCodecFrame
  */
 GstVideoCodecFrame *
 gst_video_encoder_get_oldest_frame (GstVideoEncoder * encoder)
@@ -2921,7 +2922,7 @@ gst_video_encoder_get_oldest_frame (GstVideoEncoder * encoder)
  *
  * Get a pending unfinished #GstVideoCodecFrame
  *
- * Returns: (transfer full): pending unfinished #GstVideoCodecFrame identified by @frame_number.
+ * Returns: (transfer full) (nullable): pending unfinished #GstVideoCodecFrame identified by @frame_number.
  */
 GstVideoCodecFrame *
 gst_video_encoder_get_frame (GstVideoEncoder * encoder, int frame_number)
@@ -2970,7 +2971,7 @@ gst_video_encoder_get_frames (GstVideoEncoder * encoder)
 /**
  * gst_video_encoder_merge_tags:
  * @encoder: a #GstVideoEncoder
- * @tags: (allow-none): a #GstTagList to merge, or NULL to unset
+ * @tags: (nullable): a #GstTagList to merge, or NULL to unset
  *     previously-set tags
  * @mode: the #GstTagMergeMode to use, usually #GST_TAG_MERGE_REPLACE
  *
@@ -3013,9 +3014,9 @@ gst_video_encoder_merge_tags (GstVideoEncoder * encoder,
 /**
  * gst_video_encoder_get_allocator:
  * @encoder: a #GstVideoEncoder
- * @allocator: (out) (allow-none) (transfer full): the #GstAllocator
+ * @allocator: (out) (optional) (nullable) (transfer full): the #GstAllocator
  * used
- * @params: (out) (allow-none) (transfer full): the
+ * @params: (out) (optional) (transfer full): the
  * #GstAllocationParams of @allocator
  *
  * Lets #GstVideoEncoder sub-classes to know the memory @allocator
index bfd9146..50113fa 100644 (file)
@@ -160,7 +160,7 @@ gst_video_meta_get_info (void)
  * Buffers can contain multiple #GstVideoMeta metadata items when dealing with
  * multiview buffers.
  *
- * Returns: (transfer none): the #GstVideoMeta with lowest id (usually 0) or %NULL when there
+ * Returns: (transfer none) (nullable): the #GstVideoMeta with lowest id (usually 0) or %NULL when there
  * is no such metadata on @buffer.
  */
 GstVideoMeta *
@@ -193,7 +193,7 @@ gst_buffer_get_video_meta (GstBuffer * buffer)
  * Buffers can contain multiple #GstVideoMeta metadata items when dealing with
  * multiview buffers.
  *
- * Returns: (transfer none): the #GstVideoMeta with @id or %NULL when there is no such metadata
+ * Returns: (transfer none) (nullable): the #GstVideoMeta with @id or %NULL when there is no such metadata
  * on @buffer.
  */
 GstVideoMeta *
@@ -924,7 +924,7 @@ gst_video_region_of_interest_meta_get_info (void)
  * Buffers can contain multiple #GstVideoRegionOfInterestMeta metadata items if
  * multiple regions of interests are marked on a frame.
  *
- * Returns: (transfer none): the #GstVideoRegionOfInterestMeta with @id or %NULL when there is
+ * Returns: (transfer none) (nullable): the #GstVideoRegionOfInterestMeta with @id or %NULL when there is
  * no such metadata on @buffer.
  */
 GstVideoRegionOfInterestMeta *
@@ -1178,7 +1178,7 @@ gst_buffer_add_video_time_code_meta (GstBuffer * buffer,
  * Attaches #GstVideoTimeCodeMeta metadata to @buffer with the given
  * parameters.
  *
- * Returns: (transfer none): the #GstVideoTimeCodeMeta on @buffer, or
+ * Returns: (transfer none) (nullable): the #GstVideoTimeCodeMeta on @buffer, or
  * (since 1.16) %NULL if the timecode was invalid.
  *
  * Since: 1.10
index f3ab4d8..15f06c6 100644 (file)
@@ -759,7 +759,7 @@ gst_video_time_code_new_from_date_time (guint fps_n, guint fps_d,
  * The resulting config->latest_daily_jam is set to
  * midnight, and timecode is set to the given time.
  *
- * Returns: the #GstVideoTimeCode representation of @dt, or %NULL if
+ * Returns: (nullable): the #GstVideoTimeCode representation of @dt, or %NULL if
  *   no valid timecode could be created.
  *
  * Since: 1.16
index 64d8792..1225337 100644 (file)
@@ -326,7 +326,7 @@ gst_video_vbi_parser_get_ancillary (GstVideoVBIParser * parser,
  *
  * Since: 1.16
  *
- * Returns: The new #GstVideoVBIParser or %NULL if the @format and/or @pixel_width
+ * Returns: (nullable): The new #GstVideoVBIParser or %NULL if the @format and/or @pixel_width
  * is not supported.
  */
 GstVideoVBIParser *
@@ -607,7 +607,7 @@ gst_video_vbi_encoder_free (GstVideoVBIEncoder * encoder)
  *
  * Since: 1.16
  *
- * Returns: The new #GstVideoVBIEncoder or %NULL if the @format and/or @pixel_width
+ * Returns: (nullable): The new #GstVideoVBIEncoder or %NULL if the @format and/or @pixel_width
  * is not supported.
  */
 GstVideoVBIEncoder *
index a28d0f2..aa1905d 100644 (file)
@@ -255,7 +255,7 @@ GST_VIDEO_API const GstMetaInfo *gst_video_afd_meta_get_info (void);
  *
  * Since: 1.18
  *
- * Returns: The first #GstVideoAFDMeta present on @b, or %NULL if
+ * Returns: (nullable): The first #GstVideoAFDMeta present on @b, or %NULL if
  * no #GstVideoAFDMeta are present
  */
 #define gst_buffer_get_video_afd_meta(b) \
@@ -316,7 +316,7 @@ GST_VIDEO_API const GstMetaInfo *gst_video_bar_meta_get_info (void);
  *
  * Since: 1.18
  *
- * Returns: The first #GstVideoBarMeta present on @b, or %NULL if
+ * Returns: (nullable): The first #GstVideoBarMeta present on @b, or %NULL if
  * no #GstVideoBarMeta are present
  */
 #define gst_buffer_get_video_bar_meta(b) \
@@ -407,7 +407,7 @@ const GstMetaInfo *gst_video_caption_meta_get_info (void);
  *
  * Since: 1.16
  *
- * Returns: The first #GstVideoCaptionMeta present on @b, or %NULL if
+ * Returns: (nullable): The first #GstVideoCaptionMeta present on @b, or %NULL if
  * no #GstVideoCaptionMeta are present
  */
 #define gst_buffer_get_video_caption_meta(b) \
index c64d58a..9b77eab 100644 (file)
@@ -2309,7 +2309,7 @@ convert_get_alpha_mode (GstVideoConverter * convert)
  * The optional @pool can be used to spawn threads, this is useful when
  * creating new converters rapidly, for example when updating cropping.
  *
- * Returns: a #GstVideoConverter or %NULL if conversion is not possible.
+ * Returns (nullable): a #GstVideoConverter or %NULL if conversion is not possible.
  *
  * Since: 1.20
  */
@@ -2552,7 +2552,7 @@ no_pack_func:
  * Create a new converter object to convert between @in_info and @out_info
  * with @config.
  *
- * Returns: a #GstVideoConverter or %NULL if conversion is not possible.
+ * Returns (nullable): a #GstVideoConverter or %NULL if conversion is not possible.
  *
  * Since: 1.6
  */
index 923df75..4e4ff87 100644 (file)
@@ -7664,7 +7664,7 @@ gst_video_format_get_info (GstVideoFormat format)
  * Get the default palette of @format. This the palette used in the pack
  * function for paletted formats.
  *
- * Returns: (transfer none): the default palette of @format or %NULL when
+ * Returns: (transfer none) (nullable): the default palette of @format or %NULL when
  * @format does not have a palette.
  *
  * Since: 1.2
index b3d25d3..e1fb452 100644 (file)
@@ -63,7 +63,7 @@ ensure_debug_category (void)
  *
  * Copy a GstVideoInfo structure.
  *
- * Returns: a new #GstVideoInfo. free with gst_video_info_free.
+ * Returns: (transfer full): a new #GstVideoInfo. free with gst_video_info_free.
  *
  * Since: 1.6
  */
@@ -97,7 +97,7 @@ G_DEFINE_BOXED_TYPE (GstVideoInfo, gst_video_info,
  * Allocate a new #GstVideoInfo that is also initialized with
  * gst_video_info_init().
  *
- * Returns: a new #GstVideoInfo. free with gst_video_info_free().
+ * Returns: (transfer full): a new #GstVideoInfo. free with gst_video_info_free().
  *
  * Since: 1.6
  */
@@ -295,7 +295,7 @@ static const gchar *interlace_mode[] = {
  *
  * Convert @mode to its string representation.
  *
- * Returns: @mode as a string or NULL if @mode in invalid.
+ * Returns: @mode as a string.
  *
  * Since: 1.6
  */
@@ -343,7 +343,7 @@ static const gchar *field_order[] = {
  *
  * Convert @order to its string representation.
  *
- * Returns: @order as a string or NULL if @order in invalid.
+ * Returns: @order as a string.
  *
  * Since: 1.12
  */
@@ -575,7 +575,7 @@ alternate_no_feature:
  *
  * Parse @caps to generate a #GstVideoInfo.
  *
- * Returns: A #GstVideoInfo, or %NULL if @caps couldn't be parsed
+ * Returns: (transfer full) (nullable): A #GstVideoInfo, or %NULL if @caps couldn't be parsed
  * Since: 1.20
  */
 GstVideoInfo *
@@ -656,7 +656,7 @@ gst_video_info_is_equal (const GstVideoInfo * info, const GstVideoInfo * other)
  *
  * Convert the values of @info into a #GstCaps.
  *
- * Returns: a new #GstCaps containing the info of @info.
+ * Returns: (transfer full): a new #GstCaps containing the info of @info.
  */
 GstCaps *
 gst_video_info_to_caps (const GstVideoInfo * info)
@@ -1423,7 +1423,7 @@ done:
  * gst_video_info_align_full:
  * @info: a #GstVideoInfo
  * @align: alignment parameters
- * @plane_size: (out) (allow-none): array used to store the plane sizes
+ * @plane_size: (out) (optional): array used to store the plane sizes
  *
  * Extra padding will be added to the right side when stride alignment padding
  * is required and @align will be updated with the new padding values.
index 107728d..854bf0f 100644 (file)
@@ -65,11 +65,11 @@ static const struct mview_map_t
  * gst_video_multiview_mode_to_caps_string:
  * @mview_mode: A #GstVideoMultiviewMode value
  *
- * Returns: The caps string representation of the mode, or NULL if invalid.
- *
  * Given a #GstVideoMultiviewMode returns the multiview-mode caps string
  * for insertion into a caps structure
  *
+ * Returns: (nullable): The caps string representation of the mode, or NULL if invalid.
+ *
  * Since: 1.6
  */
 const gchar *
index ee83ec1..ea5484e 100644 (file)
@@ -426,7 +426,7 @@ gst_video_overlay_composition_n_rectangles (GstVideoOverlayComposition * comp)
  *
  * Returns the @n-th #GstVideoOverlayRectangle contained in @comp.
  *
- * Returns: (transfer none): the @n-th rectangle, or NULL if @n is out of
+ * Returns: (transfer none) (nullable): the @n-th rectangle, or NULL if @n is out of
  *     bounds. Will not return a new reference, the caller will need to
  *     obtain her own reference using gst_video_overlay_rectangle_ref()
  *     if needed.
index 365b9f6..cb5d7be 100644 (file)
@@ -86,7 +86,7 @@ const GstMetaInfo *gst_video_sei_user_data_unregistered_meta_get_info (void);
  *
  * Gets the GstVideoSEIUserDataUnregisteredMeta that might be present on @b.
  *
- * Returns: The first #GstVideoSEIUserDataUnregisteredMeta present on @b, or %NULL if
+ * Returns: (nullable): The first #GstVideoSEIUserDataUnregisteredMeta present on @b, or %NULL if
  * no #GstVideoSEIUserDataUnregisteredMeta are present
  *
  * Since: 1.22