Fix edge errors in the sRGB comment; I had used subs() on the wrong expression.

Fix a typo in a comment.

Implement GammaExpansionEffect using ALU ops instead of texture lookups.

In a standalone benchmark (on a Sandy Bridge laptop), this is pretty much
a no-op performance-wise, but when more ops are put into the mix, it's
a ~20% FPS win, and but in a more real situation with multiple inputs
etc., it's subjectively also a pretty clear win. The reason is probably
that we generally are way overloaded on texture operations.

Note that we had similar code like this before (before we started using
the texture for lookup), but it used pow(), which is markedly slower than our
fourth-degree polynomial approximation.

We should probably do the same for GammaCompressionEffect.

In the demo application, reset the FPS counter every hundred frames.

Gives much, much better FPS counts for benchmarking.

Fix some outdated comments in GammaExpansionEffect.

It's now 2014 :-P (Happy new year!)

Add an assertion failure that we don't add the same effect instance twice.

Doing this causes hard-to-understand crashes, so just bomb out at once.

Calculate the RGB-to-XYZ matrix ourselves instead of using a “magic” one from Wikipedia.

Generally we try to derive such values from first principles when possible;
this is some of the oldest code in Movit, which explains why it was forgotten.

Maybe longer-term we should move this out of ColorspaceConversionEffect into a
free function in util.h or something similar, but for now, this will do.

Fix a minor error in one of the color temperature constants.

Wikipedia cites http://icpr.snu.ac.kr/resource/wop.pdf/J01/2002/041/R06/J012002041R060865.pdf,
but seems to differ in in the last decimal of one of the coefficients.
The error is, of course, inconsequential, but it's good to be correct
in any case, and I trust the paper more.

Add partial Rec. 2020 support.

Generate the FBO anew every render.

This fixes an issue where reusing the same EffectChain from two different GL
contexts would cause errors even if they are shared, because FBOs cannot be
shared between contexts (in the ARB extension, anyway).

There might be some negative performance implications to this, but I was unable
to measure any on Intel/Mesa. If this should prove to be a problem in the future,
we could maybe keep one around per context, or have the caller invalidate the
FBO for us.

Fix a bug where DeconvolutionSharpenEffect would forget one line of the kernel.

This manifested itself in that non-identity filters would start changing alpha
of solid images (since the kernel didn't sum up to one), which obviously isn't
good. Added a unit test to make sure it doesn't happen again.

Install effect_util.h; it is useful for people implementing their own effects.

Spelling fix.

Assert that the EffectChain is not finalized when someone tries to modify it.

Fix another harmless Valgrind hit.

Properly check framebuffer status when generating FBOs.

Fix an issue with invalid texture number usage in measure_texel_subpixel_precision().

A small comment fix in ColorspaceConversionEffect.

Properly link in multiply_effect.

Promote MultiplyEffect to a real effect.

The intended use is for overlays, where you'd want to do e.g.
0.2*x atop y instead of just x atop y, fading the overlay in or out.

Also, give it full RGBA inputs, as that might potentially be useful
for someone. It certainly was useful for adapting it to continue to
be used in the EffectChain unit test, at least. (It doesn't have
its own unit test, since it's so trivial.)

Fix a bug where PaddingEffect could create assertion errors.

The ratinale is explained in the comment, but in short, PaddingEffect
could convert blank to premultiplied alpha without realizing that
it then needed linear light (since premultiplied alpha in our case
is defined as being in linear light). Also added a unit test.

Reported by Christophe Thommeret.

Clip below-zero (out-of-gamut) colors in LiftGammaGainEffect.

pow(x, y) for x < 0 is undefined, and behaves differently on nVidia
and Intel. This can reasonably happen when having inputs from a different
gamut, or just a complex chain before the LGG effect; there's nothing
in Movit that prohibits out-of-sRGB-gamut colors. Thus, we need to detect
such inputs and clamp them.

We could in theory check for the special case of y=1 (no gamma change)
and allow negative values through then, but this wouldn't seem like a good
solution, especially if animating gamma.

Found and debugged by Christophe Thommeret.

Make building the demo app optional if SDL_image and/or libpng12 are not found.

Add a pkg-config variable “shaderdir” to locate the shaders.

Patch from Dan Dennedy.

Install defs.h on make install, since it is needed by effect.h.

Put the resample alpha handling on the right effect.

Indentation fix in Makefile.in.

Do not call autoheader, since we do not use config.h.

Use autoconf defaults for CXX and CXXFLAGS.

Rename RGBATriplet to RGBATuple, to avoid silliness.

Make util.h private to the effects; instead, defs.h contains what effect.h needs. Keeps some junk out of the namespace of the client.

Prefix include guards with _MOVIT to avoid clashes with external files.

Small cleanup in movit.pc.in.

Add a distclean target.

Update README for the new output alpha constants.

Fix cleaning of .dot and .frag files.

Add a pkg-config file.

Makefile should now be in .gitignore.

Remove stray file movit_pch.h, which was never meant to be checked in.

Switch to autoconf to find libraries.

Also, make a rudimentary make install command.

We don't use anything fancy from autoconf yet (in particular,
no config.h file), and we don't use automake or libtool.
Most likely, this will happen later.

Remove obsolete files header.vert and footer.vert.

Add some missing tests to .gitignore.

Stop linking unit tests et al against SDL_image and other things that only the demo program needs.

Remove an unused function. Found by Clang.

Split out some private utilities into effect_util.cpp, so we do not need to include e.g. Eigen from effect.h.

Run include-what-you-use over all of movit. Some hand tuning.

Fix alpha handling for blur; it needs to be on the working (rewritten-to) effect.

Add a new alpha handling method, INPUT_PREMULTIPLIED_ALPHA_KEEP_BLANK.

This should fix most of the problems where you only process inputs
without alpha, but the framework still inserts an AlphaDivisionEffect
at the end because it loses track of the blank alpha state throughout
the pipeline and the output expects postmultiplied alpha.

There's one important case left, though: MixEffect will always reset
tha alpha state to premultiplied. We should probably fix that too
at some later stage.

Rename the OutputAlphaFormat enums; they had gotten inconsistent after the last alpha-related rename.

Give the alpha enums somewhat better/more consistent names, and shuffle them around a bit.

In resizing effects, add the notion of a “virtual output size”.

This is the size that the effect wants to be perceived as for the next
effect in the chain, which is useful if you e.g. have a blur and then
padding. Even though the blur might rescale the image down from e.g.
512x512 to 128x128 before blurring (in case of a large blur), and this
size is what actually comes out in the texture at the other end,
it still wants to be treated as a 512x512 image when adding padding.

Reported by Christophe Thommeret.

Give SizeStoringEffect an effect_type_id(), for easier debugging of the unit test.

If all inputs to an effect have the same input size, use that instead of fitting to the aspect.

The common case here is one-input effects on overlays.
Reported by Christophe Thommeret.

Fix a Clang warning.

Patch by Dan Dennedy.

Fix a bug where intermediate phase outputs could get too low height.

Basically our aspect ratio adjustment and lack of proper roundoff
could conspire to let e.g. mix(1280x720, 939x939) = 1669x938,
which is obviously wrong. I could probably have fixed it with
an extra lrintf(), but it seems more robust to simply avoid the
extra conversion (ie., never convert height -> width -> height).

Add a unit test to verify.

Merge remote-tracking branch 'origin/master'

Allow for more than ten bits in subpixel precision measuring; modern llvmpipe seems to have 11.

DeconvolutionSharpenEffect needs texture bounce.

Anything else will be incredibly slow. Reported by Christophe Thommeret.

Fix another stack overflow in EffectChainTest.

Again, found with llvmpipe.

Loosen up a boundary in DitherEffectTest by 10%.

Seemingly needed on my nVidia machine with the newest drivers.

Fix a narrowing conversion within {} in PaddingEffect (C++11 compatibility).

Fix a stack overflow in EffectChainTest.

In OverlayEffectTest, handle that RGB on zero alpha is undefined.

Trickled by llvmpipe.

Add sdl-config --cflags and --libs when compiling.

Check return value from SDL_Init().

Use NULL instead of png_voidp_NULL.

Fixes a compile error on OS X, reportedly.

Don't override GTEST_DIR if it already is set.

Link against -lpng, since we can now write screenshots in PNG format.

Ask Google Test not to use exceptions.

Gives us slightly cleaner shutdown on error. Found by Coverity Scan.

Make the internal effects private to EffectChain.

ColorspaceConversionEffect, DitherEffect, GammaExpansionEffect and GammaCompressionEffect
are all supposed to be used by EffectChain only, so make them private; I've had
reports of users trying to use these directly, leaving the framework in a confused state.

Fix a typo in the README.

Add some parameter asserts in DitherEffect, too.

Remove two unused members from EffectChain.

Indirectly found by Coverity Scan.

Another case of NULL initialization to keep Coverity happy.

Add some asserts to SingleResamplePassEffect, to make sure the resolutions makes sense.

Optimize away duplicate conversion nodes.

Sometimes an effect can be used by two other effects that both demand
the same conversion. If so, we should simply insert a conversion after
that effect and connect _all_ outputs to that conversion, since
conversions to linear/sRGB/premultiplied never hurt for us.

Add unit tests for the gamma and colorspace cases. I could have added
for the alpha case, too, but it got very tedious. :-)

Set pixel_data[] to NULL in YCbCrInput as well.

Like the previous one, mainly for cleaner crashes.
Again, found by Coverity Scan.

Another NULL initializer, mostly to get Coverity to be quiet.

Initialize pixel_data in FlatInput to NULL.

Doesn't matter in practice, but will cause cleaner crashes if people
forget to do set_pixel_data().

Found by Coverity Scan.

Add unit tests for GlowEffect.

Clamp alpha in MixEffect.

This struck me suddenly one evening as a fix for the issues with alpha in glow,
and seems to work incredibly well for unsharp mask, too. The rationale is
outlined in the comment in the frag.

I'll probably be adding some tests for GlowEffect to make sure that it keeps
working fine, but visual tests so far look very good. The only issue is that
with destination alpha always being in linear light (as it should be),
programs that don't blend in linear light, like GIMP and ImageMagick,
get somewhat dull-looking results with e.g. glow.

Force the LC_NUMERIC locale temporarily to C in finalize(), to avoid problems with locales messing with %f.

Write PNGs instead of PPMs, so that we get destination alpha. Also, ask SDL for a GL context with destination alpha.

Add missing unit test.

Remove some leftover debugging code.

Add an effect for padding.

Add support for vec4 uniforms.

There are reports that Movit actually does work with Nouveau.

Fix another issue where an input was used twice. Add unit tests, again.

Make output_dot() cope with effects that are in multiple phases.

Add an assert saying that if an input has premultiplied alpha, it must also have linear gamma.

This holds true for nodes in general, but I'm not sure enough about all sorts of intermediate
states that could cause this to be temporarily untrue for other nodes.

Print output node in the dot graphs.

This makes it possible to look at the properties of the last
effect, which is often useful to debug the fix_output_* functions.

Add an assert to expect_equal() to make sure people send in reasonable widths and heights.

Fix a memory leak in expect_equal(). Found by Coverity Scan.

Refactor RewritingTo{MirrorEffect,InvertEffect} into a reusable template; it started getting silly with the number of Rewriting* classes.

Fix two issues related to non-treelike (diamond) effect graphs.

First of all, we could have an assert failure when an input was used twice.
Work around it by simply ignoring the input the second time.

This, however, would expose an issue where effects could be emitted in the
.glsl file out-of-order. Refactor the topological sort code so that it can
be reused for arbitrary subgraphs, and then use it to topologically sort
the list of effects in each pass.

Add a unit test to verify that all of this works.

Small spelling fix.

 Remove the code for postmultiplied alpha from OverlayEffect.

Correct atop → over in the README too.