Add a texture barrier after dispatching each compute shader.

This is maybe a bit heavy-handed (there are cases where shaders could
run in parallel), but it's by far the simplest thing to do, since we
have zero control over what happens to the textures we use when they
are handed back to the resource pool.

Add some asserts to EffectChain::execute_phase, so that we do not inadvertedly insert new output textures with number 0.

Invert the meaning of the last_phase boolean to execute_phase().

This is because we can soon have last phases that do not render to FBOs
(those that come out of compute shaders).

Use glGetTexImage() instead of glReadPixels() for reading back test data.

Generally cleaner, and also paves the way for when we have FBO-less
rendering from compute shaders.

Better error messages when skipping compute shader benchmarks.

Implement a compute shdaer version of DeinterlaceEffect.

This is currently a loss for grayscale (probably due to the extra
rgba16f bounce), but a win of about ~30% on BGRA on my Haswell.
NVIDIA doesn't care much either way.

There are some performance mysteries remaining, but it's a good start.

Reduce the clutter in instantiating benchmarks a little bit.

Add a helper class to easier test fragment and compute shader versions of the same effect alongside each other.

Fix compute shader outputs on NVIDIA.

Bump compute shader header to #version 150.

NVIDIA requires this for the layout qualifier, and it's probably right.
Note that this required moving the unit tests to a core context,
due to Mesa's demands.

Drop support for SDL1, which is no longer maintained.

Use auto and range-based for loops a few places where it aids clarity.

More renaming of .compute -> .comp.

Fix an issue where matrices would not get printed out on test failure.

Add a benchmark for DeinterlaceTest without the spatial interlacing check.

Rename .compute to .comp, to be in line with the Khronos reference compiler.

Make BGRA deinterlacing benchmark premultiplied, so that we do not use forever just in bouncing.

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).