[vlc-devel] [PATCH] [WIP] RFC: add HDR/BT.2020 support (using libplacebo)
thomas at gllm.fr
Mon Oct 30 13:18:13 CET 2017
On Fri, Oct 27, 2017, at 14:45, Niklas Haas wrote:
> On Fri, 27 Oct 2017 02:37:08 +0200, Niklas Haas <vlc at haasn.xyz> wrote:
> > On Thu, 26 Oct 2017 20:01:07 +0300, Rémi Denis-Courmont <remi at remlab.net> wrote:
> > > VLC has its own convenience wrapper for that. Obviously, that can´t be used
> > > for by an external library. But you can use a similar solution.
> > >
> > I've given the solution an attempt:
> > https://github.com/haasn/libplacebo/commit/0e67eec23628f97089bb25724248e502694c57bb
> > This works on my end, but currently only has an implementation for
> > systems with `uselocale`, which apparently Win32 does not belong to. On
> > Win32, there are locale-invariant versions of printf which could be used
> > to implement the same API, but I'm unable to produce this code. (BSD
> > could also use the printf_l etc. wrappers, although BSD supports
> > uselocale so it shouldn't be necessary)
> I've also updated my patch in order to incorporate integration with the
> build system + contrib/ (thanks to ePirat for the code), and make the
> code optional (based on #ifdef HAVE_LIBPLACEB).
> I've attached the new version.
- You should do a new commit for the contrib part.
- You should split the additions of glUniformMatrix* in the vtable in a
- The configure.ac has one error that triggers an error when placebo is
not found and not forced. It should also use AS_HELP_STRING. Here is the
Otherwise, it looks fine to me.
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> Email had 1 attachment:
> + 0001-video_output-opengl-add-basic-support-for-libplacebo.patch
> 20k (text/plain)
More information about the vlc-devel