[Bug 754120] avdec_hevc: Segfault in hevc decode with glimagesink and AVX2 CPU feature

[GStreamer (GNOME Bugzilla)]

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

--- Comment #26 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
(In reply to Sebastian Dröge (slomo) from comment #25)
> (In reply to Tim-Philipp Müller from comment #24)
> > Olivier suggested on IRC to move the loop that checks the alignments into a
> > utility function. This means a caller would still have to do get_allocator()
> > to get the AllocationParams and then call set_allocator() again if it was
> > updated. It's the most transparent way of doing things though.
> 
> Is it going to be needed in many places (more than 2) that it makes sense to
> have an utility function for it?

And with that I mean, a utility function doesn't seem very useful for this.
Code that uses the videoalignment API has to set the separate stride aligns
anyway, it could as well just set the allocation parameters accordingly then. I
don't think a utility function will make much of a difference here.

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