Parametrize BM_DeinterlaceEffect so that we have both grayscale and BGRA.

Fix deletion of compute shaders.

Fix some missing NULL -> nullptr.

Revert "Remove C++11 dependency from ResampleEffect."

Add back unique_ptr, now that we have C++11.

This reverts commit 8e9f58fec54a4c879035b214fd7411f6ff7b3a32.

Use double right angle brackets for nested templates everywhere, now that we have C++11.

Use nullptr everywhere, now that we have C++11.

Add support for microbenchmarks.

This depends on Google's microbenchmark library (latest git version,
at least as soon as they pull in the pkg-config file), but it is
not mandatory.

Note that this makes Movit depend on C++11.

Add support for compute shaders.

This is currently pretty raw, and there are no effect using it yet,
but it's a useful first step for getting the infrastructure in place.

There's no need to #undef PREFIX, since we do the token pasting ourselves.

Fix some invalid code in DeinterlaceTest; GCC 7 would optimize it into simply an infinite loop.

Some refactoring in ResourcePool.

Add the GPLv2 license text.

fix compile error: "error: variable-sized object ‘red_data’ may not be initialized"

Support negative values for lift in LiftGammaGainEffect.

Patch by Jean-Baptiste Mardelle <jb@kdenlive.org> (except the test,
which is by myself). Based on work from Dušan Hanuš <hanus@pixelhouse.cz>.

Fix a typo.

Release Movit 1.5.3.

Add a notice to render_to_screen() that it might be suboptimal.

Fix a leak if ResampleEffect is destroyed before being put on a chain.

FFTConvolutionEffect remembered this, but ResampleEffect did not.
Patch by Dan Dennedy.

Fix fp16_test compilation _without_ f16c-enabled compilation flags.

Reported by Arthur Huillet.

Fix fp16_test compilation with f16c-enabled compilation flags.

Remove C++11 dependency from ResampleEffect.

We really ought to allow C++11 in Movit soon, but 1.5.2 is not the right release.

Reported by Etienne d'Hautefeuille.

Fix a double-unlock.

Release Movit 1.5.2.

Add some room for future expansion in ResampleEffect, since I have something that I would like them for and we are breaking the ABI anyway.

Cache VAOs between runs.

NVIDIA's drivers (at least the Linux drivers) seem to have a bug
where, if you call glVertexAttribPointer() in one thread together
with some specific activity in another thread (at least another
glVertexAttribPointer() will do, but I've also seen it during
glXSwapBuffers()), it will try to free a non-malloced pointer,
crashing everything. This happens very rarely, but often enough
that it's a real problem for Nageru.

We solve it by simply pre-creating all needed VAOs ahead-of-time
and caching them, which makes us call glVertexAttribPointer()
a _lot_ less often (basically never, instead of several times
per frame). This might actually be a bit slower than the old code
(I haven't tested), but at least it works around the driver bug.

ABI break, unfortunately, but no API break.

Fix Makefile after fp16 removal.

Inline the fp16 conversion routines. Helps 18% on ResampleEffect::calculate_texture.

Inline combine_two_samples (and remove an obsolete assert). Helps 13–14% on ResampleEffect::calculate_texture.

Some microoptimizations in combine_two_samples(). Saves about 4% in ResampleEffect::calculate_weights.

Remove some unneeded conversions from ResampleEffect. Speeds up texture generation by ~8%.

In ResampleEffect, split the weight calculation out into its own function that does not depend on OpenGL.

Release Movit 1.5.1. (No ABI break, as only a new non-virtual member function is added.)

Fix multiply-defined symbols with multiple YCbCrInputs in the same chain.

Allow YCbCrInput to change input format after finalize.

The performance should be very nearly identical; only a little
possible constant folding, and that's it. There's only one
(documented) limitation, and it's both rare and should be easy
to work around.

In YCbCrInput, fix an issue where offsets would be treated as equal even if their y value differed.

Add a warning on FlatInput::set_texture_num() about sRGB behavior.

Release Movit 1.5.0, including a small README update.

Loosen a test bound slightly for the benefit of NVIDIA cards.

Add an explanatory comment about dither with 16-bit output.

Support 10-/12-bit Y'CbCr output packed in 16-bit.

This mirrors our existing input support, and makes planar and
semiplanar output possible for 10-/12-bit Y'CbCr.

ABI break. API stays the same.

Don't try to check Cb and Cr positioning for output formats, as they should be non-subsampled anyway.

Support multiple Y'CbCr outputs.

This is rather limited for now, unfortunately, but it solves a
rather specific problem in Nageru.

Update another outdated comment.

Update an outdated comment.

Loosen up the accuracy bounds a tiny bit in some tests for NVIDIA cards, which have worse rounding (we do not bother to add manual rounding for these cards, since the difference is so small).

Fix an issue where the last pass would have been rendered with the sRGB flag set, which confused Qt applications running in certain NVIDIA configurations.

Fix compiling without C++11.

ABI break. Reported by Dan Dennedy.

Fix a bad typo.

Bump MOVIT_VERSION for all the YCbCr changes.

Treat num_levels == 0 as 256, for the benefit of older applications.

Cosmetic tweak in YCbCrInput.

Fix a stack buffer overrun in ycbcr_input_test.

Implement mipmap generation in YCbCrInput, now that we advertise single-texture.

Loosen up some restrictions on YCbCrInput if we have interleaved mode.

Add support for 10- and 12-bit planar Y'CbCr inputs.

This is mostly for completeness; at least for 10-bit, 10:10:10:2
should be a faster format. However, it's nice to allow direct
subsampled inputs _somehow_.

Some minor comment fixes in ycbcr.h.

Add input support for packed 10-bit Y'CbCr.

Support interleaved (chunky) 4:4:4 in YCbCrInput.

Fix some test breakage.

Support 10- and 12-bit Y'CbCr output.

We don't have any input support yet; the constants are put in place,
but we also need some work on supporting (semi-)adequate input formats.

Allow adjusting the output Y'CbCr coefficients after finalize.

Primarily useful for Nageru, which may have to switch output modes runtime.
Pretty much the same speed (just a single extra branch on a boolean uniform),
as constants and uniforms are typically the same speed and we're generally
ALU-bound.

Release Movit 1.4.0.

Fix a typo.

Fix an issue where a (cached) shader program could be used from multiple
threads at a time.

This isn't allowed, since uniforms belong to the program, not to the
context. Found by running Helgrind over Nageru.

Make the error printed on check_error() slightly friendlier: Include the enum if possible, and print it to stderr instead of stdout.

Be more defensive about width/height/pitch given to FlatInput and YCbCrInput.

Remove a very old and outdated comment.

Add some clarifying comments about the intermediate formats.

Fix some comment formatting.

Rework PaddingEffect alpha handling, which also fixes a long-standing assertion failure.

Allow storing values in intermediate framebuffers as sqrt(x).

Together with GL_RGB10_A2, this would seem to be an even better tradeoff for
many chains than GL_SRGB8_ALPHA8 is, as long as you don't need intermediate
alpha. (We verify its accuracy with a unit test.)

This changes the API for specifying intermediate framebuffers, but that API
was never in a release, so it should be fine.

Also document a rather obscure problem where, if you can actually hold on to
non-linear values across a bounce buffer, you don't really want to store them
in sRGB encoding. (The square-root version actually avoids this problem.
I guess we could snoop on the type and do a similar thing if we see it's an
GL_SRGB* encoding, but it seems so obscure that we can ignore it for now.)

Hard-assert on something that has bitten me too many times now.

Fix a typo.

Add support for 10-bit RGB framebuffers.

Add some sRGB conversion functions to test_util.{h,cpp}.

Bump version after new format support.

Add support for some of the more esoteric minifloat formats to ResourcePool.

Do not send NULL to glTexSubImage2D if there is no input data set; it is illegal, and NVIDIA's drivers crash on it.

Merge branch '1.3.x-release'

Release Movit 1.3.2. (From a branch, since I do not want to break ABI compatibility at this point.)

Make the sRGB intermediate test slightly more stringent (so that e.g. 0.0 will not work).

Make sRGBIntermediate test less sensitive to the exact sRGB choices; fixes unit test on NVIDIA.

Require OpenGL 3.0 unconditionally; this is a no-op, since we already required GLSL 1.30 (part of OpenGL 3.0).

Allow setting the intermediate texture format; useful for reducing bandwidth at the expense of quality, and possibly future GLES2 support.

Make timer query objects polled asynchronously, so that the CPU blocks less on the GPU when doing timing. ABI break.

Remove an extern definition that no longer exists.

Correct/update a comment on rounding.

Make gamma polynomial constants into an array; slightly fewer uniforms to set, and it makes sense overall, since they belong so much together.

Fix initialisation on locale with comma as numerical separator

Release Movit 1.3.1.

Add deinterlace_effect_test from .gitignore. Reported by Christophe Thommeret.

2016 README updates.

Remove GL_GLEXT_PROTOTYPES from some files, since it is irrelevant with epoxy.

Remove the check for movit_shader_rounding_supported, as we now demand 1.30 unconditionally.

Do not bother with unbinding vertex attributes; that is automatically done when we unbind the VAO.

Remove a few unneeded shader program switches.

Optimize VAO/VBO usage for minimal state changes.

This is similar to what we had earlier to just reuse the VAO,
but now with correct bindings no matter what vertex attributes
are assigned to what index, so that the (new) test passes.

This is actually slightly more efficient than what we had before,
since we don't look up the attributes by text anymore, and don't
reupload the VBO for each frame anymore. In practice, the effects
should be small.

Revert "Reuse the VAO across all phases."

The patch trickles a bug where, if the first phase doesn't need texture
coordinates, the rest of the phases don't get it either. (Or more generally,
if the vertex shader varying indices are not predictable, the patch does
the wrong thing.) Add a unit test and revert it for now; in time, we'll find a
way that's both low-overhead (the patch fixes a real problem) _and_ correct in
these cases.

This reverts patch 5e34f7a8969f4afc169f034d34fb908019b3a389.

Reported by Christophe Thommeret.

Release Movit 1.3.0.

In OverlayEffect, add support for swapping the inputs.