[Bug 764873] gldeinterlace: enable this plugin on OpenGL ES using a simple deinterlace fragment shader

[GStreamer (GNOME Bugzilla)]

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

Matthew Waters (ystreet00) <ystreet00 at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #325707|none                        |reviewed
             status|                            |

--- Comment #5 from Matthew Waters (ystreet00) <ystreet00 at gmail.com> ---
Review of attachment 325707:
 --> (https://bugzilla.gnome.org/review?bug=764873&attachment=325707)

I think this should be behind a property, say method, like the deinterlace
element in -good.  There's nothing technically prohibiting enabling this
(vertical blur) shader (and the existing greedyh one everywhere).  In fact, all
that's required is to add a property and merge the two _callback()
implementations.

::: ext/gl/gstgldeinterlace.c
@@ +200,3 @@

+  GST_GL_BASE_FILTER_CLASS (klass)->supported_gl_api =
+      GST_GL_API_OPENGL | GST_GL_API_GLES2;

gles2 shaders are also supported for OpenGL3.

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