videopool: update video alignment after applying
authorSong Bing <b06498@freescale.com>
Mon, 22 Dec 2014 14:25:04 +0000 (09:25 -0500)
committerNicolas Dufresne <nicolas.dufresne@collabora.co.uk>
Mon, 22 Dec 2014 14:25:04 +0000 (09:25 -0500)
Video buffer pool will update video alignment to respect stride alignment
requirement. But haven't updated it to video alignment in configure.
Which will cause user get wrong video alignment.

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

gst-libs/gst/video/gstvideopool.c
sys/ximage/ximagepool.c
sys/xvimage/xvimagepool.c

index abc6ba5..71bd562 100644 (file)
@@ -169,6 +169,7 @@ video_buffer_pool_set_config (GstBufferPool * pool, GstStructure * config)
     /* get an apply the alignment to the info */
     gst_buffer_pool_config_get_video_alignment (config, &priv->video_align);
     gst_video_info_align (&info, &priv->video_align);
+    gst_buffer_pool_config_set_video_alignment (config, &priv->video_align);
   }
   priv->info = info;
 
index 3582f11..6a0d123 100644 (file)
@@ -582,6 +582,8 @@ ximage_buffer_pool_set_config (GstBufferPool * pool, GstStructure * config)
     /* do padding and alignment */
     gst_video_info_align (&info, &xpool->align);
 
+    gst_buffer_pool_config_set_video_alignment (config, &xpool->align);
+
     /* we need the video metadata too now */
     xpool->add_metavideo = TRUE;
   } else {
index 614f197..a113f2f 100644 (file)
@@ -106,6 +106,8 @@ xvimage_buffer_pool_set_config (GstBufferPool * pool, GstStructure * config)
     /* do padding and alignment */
     gst_video_info_align (&info, &xvpool->align);
 
+    gst_buffer_pool_config_set_video_alignment (config, &xvpool->align);
+
     /* we need the video metadata too now */
     xvpool->add_metavideo = TRUE;
   } else {