(In reply to seppo.yli-olli from comment #33)
> (In reply to Edwin Flores [:eflores] [:edwin] from comment #29)
> > * gst_buffer_pool_config_get_video_alignment returns total rubbish, even
> > when the option GST_BUFFER_POOL_OPTION_VIDEO_ALIGNMENT is present. For now,
> > gst_video_info_align is commented out.
> I'm no expert but shouldn't you check that return value of
> gst_buffer_pool_config_get_video_alignment is TRUE before using what it
> produces? Based on quick reading of gst looks like if it doesn't return
> TRUE, it failed to parse config.
Yeah. Last weekend I rebased the patch and fixed this and a few more issues (OMTC broke the patch apparently). I will post an update soonish.
(In reply to seppo.yli-olli from comment #33) pool_config_ get_video_ alignment returns total rubbish, even POOL_OPTION_ VIDEO_ALIGNMENT is present. For now, info_align is commented out. pool_config_ get_video_ alignment is TRUE before using what it
> (In reply to Edwin Flores [:eflores] [:edwin] from comment #29)
> > * gst_buffer_
> > when the option GST_BUFFER_
> > gst_video_
> I'm no expert but shouldn't you check that return value of
> gst_buffer_
> produces? Based on quick reading of gst looks like if it doesn't return
> TRUE, it failed to parse config.
Yeah. Last weekend I rebased the patch and fixed this and a few more issues (OMTC broke the patch apparently). I will post an update soonish.