[Bug 689191] d3dvideosink improvements

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=689191
  GStreamer | gst-plugins-bad | 0.10.x

--- Comment #17 from David Hoyt <dhoyt at hoytsoft.org> 2013-01-10 10:38:42 UTC ---
(In reply to comment #16)
> Ah now I finally understand what you mean. Well, if the plugin fails to load
> because of missing references it will be ignored by the registry. No problem
> there at all then :)

Still have the added benefit of doing runtime analysis of the best version to
load/use. :D IIRC even though later versions are backwards compatible, that
compatibility comes at a slight cost -- that is, certain operations that would
be more performant with later versions require extra work to be compatible w/
v9 causing a hit (likely small). But it's been a long time since I read up on
that so my memory could be off.

I should also add that later versions do diverge in several ways in their
approach and even the solution I had provided might require further tweaking --
but at least it's a start. :D

> Mostly because of the (from my understanding) unneeded code complexity (and
> thus maintenance overhead), and complexity during the build. Now this just uses
> the system Direct3D headers and properly links to everything (thus failing
> properly during link time, and during library load time).

It really wasn't that complex. The old code used the system Direct3D headers
and would fail to compile if they weren't present. IIRC, I didn't implement the
DX10/11 b/c they were backwards compatible w/ 9 but also b/c mingw-w64 didn't
have good (or any), usable DX10/11 headers.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.