From 57b9ea8f54ebabe2cbb9bb4ca2297f1d49e69187 Mon Sep 17 00:00:00 2001 From: Wim Taymans Date: Thu, 6 Nov 2014 09:44:14 +0100 Subject: [PATCH] video-format: update some docs and add a FIXME(2.0) --- gst-libs/gst/video/video-format.h | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/gst-libs/gst/video/video-format.h b/gst-libs/gst/video/video-format.h index e51c70df06..482a36c9b0 100644 --- a/gst-libs/gst/video/video-format.h +++ b/gst-libs/gst/video/video-format.h @@ -272,14 +272,18 @@ typedef void (*GstVideoFormatUnpack) (const GstVideoFormatInfo *info, * contain at least pack_lines lines with a stride of @sstride and @y * should be a multiple of pack_lines. * - * Subsampled formats will use the horizontally cosited component in the - * destination. Subsampling should be performed before packing. + * Subsampled formats will use the horizontally and vertically cosited + * component from the source. Subsampling should be performed before + * packing. * - * Because tis function does not have a x coordinate, it is not possible to + * Because this function does not have a x coordinate, it is not possible to * pack pixels starting from an unaligned position. For tiled images this * means that packing should start from a tile coordinate. For subsampled * formats this means that a complete pixel need to be packed. */ +/* FIXME(2.0): remove the chroma_site, it is unused and is not relevant for + * packing, chroma subsampling based on chroma-site should be done in a separate + * step before packing*/ typedef void (*GstVideoFormatPack) (const GstVideoFormatInfo *info, GstVideoPackFlags flags, const gpointer src, gint sstride,