Run include-what-you-use over all of movit. Some hand tuning.
authorSteinar H. Gunderson <sgunderson@bigfoot.com>
Sat, 2 Feb 2013 13:11:29 +0000 (14:11 +0100)
committerSteinar H. Gunderson <sgunderson@bigfoot.com>
Sat, 2 Feb 2013 13:11:29 +0000 (14:11 +0100)
71 files changed:
alpha_division_effect.h
alpha_division_effect_test.cpp
alpha_multiplication_effect.h
alpha_multiplication_effect_test.cpp
blur_effect.cpp
blur_effect.h
blur_effect_test.cpp
colorspace_conversion_effect.cpp
colorspace_conversion_effect.h
colorspace_conversion_effect_test.cpp
deconvolution_sharpen_effect.cpp
deconvolution_sharpen_effect.h
deconvolution_sharpen_effect_test.cpp
demo.cpp
diffusion_effect.cpp
diffusion_effect.h
diffusion_effect_test.cpp
dither_effect.cpp
dither_effect.h
dither_effect_test.cpp
effect.cpp
effect.h
effect_chain.cpp
effect_chain.h
effect_chain_test.cpp
flat_input.h
flat_input_test.cpp
gamma_compression_effect.h
gamma_compression_effect_test.cpp
gamma_expansion_effect.h
gamma_expansion_effect_test.cpp
glow_effect.cpp
glow_effect.h
glow_effect_test.cpp
gtest_sdl_main.cpp
init.cpp
lift_gamma_gain_effect.cpp
lift_gamma_gain_effect.h
lift_gamma_gain_effect_test.cpp
mirror_effect.h
mix_effect.h
mix_effect_test.cpp
overlay_effect.h
overlay_effect_test.cpp
padding_effect.cpp
padding_effect.h
padding_effect_test.cpp
resample_effect.cpp
resample_effect.h
resample_effect_test.cpp
resize_effect.h
sandbox_effect.cpp
sandbox_effect.h
saturation_effect.h
saturation_effect_test.cpp
test_util.cpp
test_util.h
unsharp_mask_effect.cpp
unsharp_mask_effect.h
unsharp_mask_effect_test.cpp
util.cpp
util.h
vignette_effect.cpp
vignette_effect.h
white_balance_effect.cpp
white_balance_effect.h
white_balance_effect_test.cpp
widgets.cpp
ycbcr_input.cpp
ycbcr_input.h
ycbcr_input_test.cpp

index e700112..2b51c3a 100644 (file)
@@ -3,6 +3,8 @@
 
 // Convert premultiplied alpha to postmultiplied alpha, simply by multiplying.
 
+#include <string>
+
 #include "effect.h"
 
 class AlphaDivisionEffect : public Effect {
index 8c29702..b9c16b3 100644 (file)
@@ -1,7 +1,9 @@
 // Unit tests for AlphaDivisionEffect.
 
-#include "test_util.h"
+#include <GL/glew.h>
 #include "gtest/gtest.h"
+#include "image_format.h"
+#include "test_util.h"
 
 TEST(AlphaDivisionEffectTest, SimpleTest) {
        const int size = 2;
index c64f2dc..2425a42 100644 (file)
@@ -3,6 +3,8 @@
 
 // Convert postmultiplied alpha to premultiplied alpha, simply by multiplying.
 
+#include <string>
+
 #include "effect.h"
 
 class AlphaMultiplicationEffect : public Effect {
index 6ee4289..62863e9 100644 (file)
@@ -1,7 +1,11 @@
 // Unit tests for AlphaMultiplicationEffect.
 
-#include "test_util.h"
+#include <GL/glew.h>
+
+#include "effect_chain.h"
 #include "gtest/gtest.h"
+#include "image_format.h"
+#include "test_util.h"
 
 TEST(AlphaMultiplicationEffectTest, SimpleTest) {
        const int size = 3;
index d62b056..a74cd41 100644 (file)
@@ -1,6 +1,7 @@
-#include <math.h>
-#include <assert.h>
 #include <GL/glew.h>
+#include <assert.h>
+#include <math.h>
+#include <algorithm>
 
 #include "blur_effect.h"
 #include "effect_chain.h"
index 76f7389..5a04415 100644 (file)
@@ -9,8 +9,15 @@
 // which is what the user is intended to use, instantiates two copies of
 // SingleBlurPassEffect behind the scenes).
 
+#include <GL/glew.h>
+#include <assert.h>
+#include <stddef.h>
+#include <string>
+
 #include "effect.h"
 
+class EffectChain;
+class Node;
 class SingleBlurPassEffect;
 
 class BlurEffect : public Effect {
index d458e45..1718e7d 100644 (file)
@@ -1,9 +1,12 @@
 // Unit tests for BlurEffect.
 #include <math.h>
+#include <string.h>
 
-#include "test_util.h"
-#include "gtest/gtest.h"
 #include "blur_effect.h"
+#include "effect_chain.h"
+#include "gtest/gtest.h"
+#include "image_format.h"
+#include "test_util.h"
 
 TEST(BlurEffectTest, IdentityTransformDoesNothing) {
        const int size = 4;
index 3a8ad95..5406e1b 100644 (file)
@@ -1,10 +1,10 @@
 #include <assert.h>
-
+#include <Eigen/Core>
 #include <Eigen/LU>
 
 #include "colorspace_conversion_effect.h"
-#include "util.h"
 #include "d65.h"
+#include "util.h"
 
 using namespace Eigen;
 
index 15ad0ad..b1d6e06 100644 (file)
@@ -8,8 +8,11 @@
 // We don't do any fancy gamut mapping or similar; colors that are out-of-gamut
 // will simply stay out-of-gamut, and probably clip in the output stage.
 
+#include <string>
+
 #include "effect.h"
 #include "effect_chain.h"
+#include "image_format.h"
 
 class ColorspaceConversionEffect : public Effect {
 private:
index 3361489..76bcd49 100644 (file)
@@ -1,8 +1,10 @@
 // Unit tests for ColorspaceConversionEffect.
 
-#include "test_util.h"
-#include "gtest/gtest.h"
+#include <GL/glew.h>
+
 #include "colorspace_conversion_effect.h"
+#include "gtest/gtest.h"
+#include "test_util.h"
 
 TEST(ColorspaceConversionEffectTest, Reversible) {
        float data[] = {
index 2d32ecd..2602664 100644 (file)
@@ -2,12 +2,15 @@
 // Since all of our signals are symmetrical, discrete correlation and convolution
 // is the same operation, and so we won't make a difference in notation.
 
-
-#include <math.h>
-#include <assert.h>
-#include <GL/glew.h>
 #include <Eigen/Dense>
 #include <Eigen/Cholesky>
+#include <GL/glew.h>
+#include <assert.h>
+#include <math.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <algorithm>
+#include <new>
 
 #include "deconvolution_sharpen_effect.h"
 #include "util.h"
index a0b405a..80553ad 100644 (file)
 //
 //   Jain, Anil K.: “Fundamentals of Digital Image Processing”, Prentice Hall, 1988.
 
-#include "effect.h"
-
+#include <GL/glew.h>
 #include <Eigen/Dense>
+#include <string>
+
+#include "effect.h"
 
 class DeconvolutionSharpenEffect : public Effect {
 public:
index f08225b..6980b3d 100644 (file)
@@ -1,8 +1,13 @@
 // Unit tests for DeconvolutionSharpenEffect.
 
-#include "test_util.h"
-#include "gtest/gtest.h"
+#include <math.h>
+#include <stdlib.h>
+
 #include "deconvolution_sharpen_effect.h"
+#include "effect_chain.h"
+#include "gtest/gtest.h"
+#include "image_format.h"
+#include "test_util.h"
 
 TEST(DeconvolutionSharpenEffectTest, IdentityTransformDoesNothing) {
        const int size = 4;
index 7b31528..0083f68 100644 (file)
--- a/demo.cpp
+++ b/demo.cpp
@@ -4,33 +4,37 @@
 #define WIDTH 1280
 #define HEIGHT 720
 
-#include <string.h>
-#include <math.h>
-#include <time.h>
-#include <sys/time.h>
-#include <assert.h>
-
-#include <string>
-#include <vector>
-#include <map>
-
 #include <GL/glew.h>
-
 #include <SDL/SDL.h>
-#include <SDL/SDL_opengl.h>
+#include <SDL/SDL_error.h>
+#include <SDL/SDL_events.h>
 #include <SDL/SDL_image.h>
+#include <SDL/SDL_keyboard.h>
+#include <SDL/SDL_keysym.h>
+#include <SDL/SDL_mouse.h>
+#include <SDL/SDL_video.h>
+#include <assert.h>
+#include <features.h>
+#include <math.h>
 #include <png.h>
+#include <pngconf.h>
+#include <setjmp.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include <sys/time.h>
+#include <time.h>
 
-#include "init.h"
+#include "diffusion_effect.h"
 #include "effect.h"
 #include "effect_chain.h"
-#include "util.h"
-#include "widgets.h"
-
 #include "flat_input.h"
+#include "image_format.h"
+#include "init.h"
 #include "lift_gamma_gain_effect.h"
 #include "saturation_effect.h"
-#include "diffusion_effect.h"
+#include "util.h"
+#include "widgets.h"
 
 unsigned char result[WIDTH * HEIGHT * 4];
 
index f07f9d1..a202f8c 100644 (file)
@@ -1,8 +1,8 @@
-#include <math.h>
 #include <assert.h>
+#include <vector>
 
-#include "diffusion_effect.h"
 #include "blur_effect.h"
+#include "diffusion_effect.h"
 #include "effect_chain.h"
 #include "util.h"
 
index 2c635d0..64d743d 100644 (file)
 // where we first blur the picture, and then overlay it on the original
 // using the original as a matte.
 
+#include <GL/glew.h>
+#include <assert.h>
+#include <string>
+
 #include "effect.h"
 
 class BlurEffect;
+class EffectChain;
+class Node;
 class OverlayMatteEffect;
 
 class DiffusionEffect : public Effect {
index 446a51f..5558f4f 100644 (file)
@@ -1,8 +1,10 @@
 // Unit tests for DiffusionEffect.
 
-#include "test_util.h"
-#include "gtest/gtest.h"
 #include "diffusion_effect.h"
+#include "effect_chain.h"
+#include "gtest/gtest.h"
+#include "image_format.h"
+#include "test_util.h"
 
 TEST(DiffusionEffectTest, IdentityTransformDoesNothing) {
        const int size = 4;
index 305268e..44933b6 100644 (file)
@@ -1,6 +1,6 @@
-#include <math.h>
-#include <assert.h>
 #include <GL/glew.h>
+#include <assert.h>
+#include <algorithm>
 
 #include "dither_effect.h"
 #include "util.h"
index cad6488..3143907 100644 (file)
@@ -43,6 +43,9 @@
 // like many LCD monitors do, but it starts to get very hairy, again, for limited gains.)
 // The dither is also deterministic across runs.
 
+#include <GL/glew.h>
+#include <string>
+
 #include "effect.h"
 
 class DitherEffect : public Effect {
index bdba121..6fd46db 100644 (file)
@@ -1,9 +1,12 @@
 // Unit tests for DitherEffect.
 
+#include <GL/glew.h>
 #include <math.h>
 
-#include "test_util.h"
+#include "effect_chain.h"
 #include "gtest/gtest.h"
+#include "image_format.h"
+#include "test_util.h"
 
 TEST(DitherEffectTest, NoDitherOnExactValues) {
        const int size = 4;
index 39e9490..0fe09db 100644 (file)
@@ -1,10 +1,11 @@
+#include <Eigen/Core>
+#include <GL/glew.h>
+#include <assert.h>
 #include <stdio.h>
 #include <string.h>
-#include <assert.h>
-#include <GL/glew.h>
+#include <utility>
 
 #include "effect.h"
-#include "effect_chain.h"
 #include "util.h"
 
 GLint get_uniform_location(GLuint glsl_program_num, const std::string &prefix, const std::string &key)
index 771648c..826dd6f 100644 (file)
--- a/effect.h
+++ b/effect.h
 // effect instance; use the macro PREFIX() around your identifiers to
 // automatically prepend that prefix.
 
+#include <GL/glew.h>
+#include <assert.h>
+#include <stddef.h>
+#include <Eigen/Core>
 #include <map>
 #include <string>
 #include <vector>
 
-#include <assert.h>
-
-#include <Eigen/Core>
-
-#include <GL/glew.h>
 #include "util.h"
 
 class EffectChain;
index 0f8876d..0015197 100644 (file)
@@ -1,27 +1,29 @@
 #define GL_GLEXT_PROTOTYPES 1
 
-#include <stdio.h>
+#include <GL/glew.h>
+#include <assert.h>
+#include <locale.h>
 #include <math.h>
+#include <stddef.h>
+#include <stdio.h>
+#include <stdlib.h>
 #include <string.h>
-#include <locale.h>
-#include <assert.h>
-#include <GL/glew.h>
-
 #include <algorithm>
 #include <set>
 #include <stack>
 #include <vector>
 
-#include "util.h"
-#include "effect_chain.h"
-#include "gamma_expansion_effect.h"
-#include "gamma_compression_effect.h"
-#include "colorspace_conversion_effect.h"
-#include "alpha_multiplication_effect.h"
 #include "alpha_division_effect.h"
+#include "alpha_multiplication_effect.h"
+#include "colorspace_conversion_effect.h"
 #include "dither_effect.h"
-#include "input.h"
+#include "effect.h"
+#include "effect_chain.h"
+#include "gamma_compression_effect.h"
+#include "gamma_expansion_effect.h"
 #include "init.h"
+#include "input.h"
+#include "util.h"
 
 EffectChain::EffectChain(float aspect_nom, float aspect_denom)
        : aspect_nom(aspect_nom),
index f5e27c8..2199a5d 100644 (file)
@@ -1,14 +1,20 @@
 #ifndef _EFFECT_CHAIN_H
 #define _EFFECT_CHAIN_H 1
 
+#include <GL/glew.h>
+#include <stdio.h>
+#include <map>
 #include <set>
+#include <string>
 #include <vector>
 
 #include "effect.h"
 #include "image_format.h"
 #include "input.h"
 
+class Effect;
 class EffectChain;
+class Input;
 struct Phase;
 
 // For internal use within Node.
index cea22e0..49d431b 100644 (file)
@@ -3,13 +3,18 @@
 // Note that this also contains the tests for some of the simpler effects.
 
 #include <GL/glew.h>
+#include <assert.h>
+#include <stddef.h>
 
+#include "effect.h"
 #include "effect_chain.h"
 #include "flat_input.h"
 #include "gtest/gtest.h"
+#include "input.h"
 #include "mirror_effect.h"
 #include "resize_effect.h"
 #include "test_util.h"
+#include "util.h"
 
 TEST(EffectChainTest, EmptyChain) {
        float data[] = {
index 24a3fca..80e113f 100644 (file)
@@ -1,10 +1,14 @@
 #ifndef _FLAT_INPUT_H
 #define _FLAT_INPUT_H 1
 
+#include <GL/glew.h>
 #include <assert.h>
+#include <string>
 
-#include "input.h"
+#include "effect.h"
+#include "image_format.h"
 #include "init.h"
+#include "input.h"
 
 // A FlatInput is the normal, “classic” case of an input, where everything
 // comes from a single 2D array with chunky pixels.
index db03bd0..c8df483 100644 (file)
@@ -1,8 +1,11 @@
 // Unit tests for FlatInput.
 
-#include "test_util.h"
-#include "gtest/gtest.h"
+#include <stddef.h>
+
+#include "effect_chain.h"
 #include "flat_input.h"
+#include "gtest/gtest.h"
+#include "test_util.h"
 
 TEST(FlatInput, SimpleGrayscale) {
        const int size = 4;
index 46ac80e..f34bca0 100644 (file)
@@ -7,8 +7,11 @@
 //
 // Currently supports sRGB and Rec. 601/709.
 
+#include <string>
+
 #include "effect.h"
 #include "effect_chain.h"
+#include "image_format.h"
 
 #define COMPRESSION_CURVE_SIZE 4096
 
index 6211acc..c0b4ccf 100644 (file)
@@ -3,9 +3,10 @@
 // Pretty much the inverse of the GammaExpansionEffect tests;
 // EffectChainTest tests that they are actually inverses.
 
-#include "test_util.h"
+#include <GL/glew.h>
 #include "gtest/gtest.h"
-#include "gamma_expansion_effect.h"
+#include "image_format.h"
+#include "test_util.h"
 
 TEST(GammaCompressionEffectTest, sRGB_KeyValues) {
        float data[] = {
index 043731b..059b246 100644 (file)
@@ -7,8 +7,11 @@
 //
 // Currently supports sRGB and Rec. 601/709.
 
+#include <string>
+
 #include "effect.h"
 #include "effect_chain.h"
+#include "image_format.h"
 
 #define EXPANSION_CURVE_SIZE 256
 
index 94f5749..6fc0252 100644 (file)
@@ -1,8 +1,10 @@
 // Unit tests for GammaExpansionEffect.
 
-#include "test_util.h"
-#include "gtest/gtest.h"
+#include <GL/glew.h>
+
 #include "gamma_expansion_effect.h"
+#include "gtest/gtest.h"
+#include "test_util.h"
 
 TEST(GammaExpansionEffectTest, sRGB_KeyValues) {
        float data[] = {
index 064cdac..86a3d7f 100644 (file)
@@ -1,10 +1,10 @@
-#include <math.h>
 #include <assert.h>
+#include <vector>
 
-#include "glow_effect.h"
 #include "blur_effect.h"
-#include "mix_effect.h"
 #include "effect_chain.h"
+#include "glow_effect.h"
+#include "mix_effect.h"
 #include "util.h"
 
 GlowEffect::GlowEffect()
index abadb84..3453c99 100644 (file)
@@ -4,11 +4,17 @@
 // Glow: Cut out the highlights of the image (everything above a certain threshold),
 // blur them, and overlay them onto the original image.
 
+#include <GL/glew.h>
+#include <assert.h>
+#include <string>
+
 #include "effect.h"
 
 class BlurEffect;
-class MixEffect;
+class EffectChain;
 class HighlightCutoffEffect;
+class MixEffect;
+class Node;
 
 class GlowEffect : public Effect {
 public:
index 77c9d12..b8eff4b 100644 (file)
@@ -2,9 +2,11 @@
 
 #include <math.h>
 
-#include "test_util.h"
-#include "gtest/gtest.h"
+#include "effect_chain.h"
 #include "glow_effect.h"
+#include "gtest/gtest.h"
+#include "image_format.h"
+#include "test_util.h"
 
 TEST(GlowEffectTest, NoAmountDoesNothing) {
        const int size = 4;
index fa3a835..39027e6 100644 (file)
@@ -1,6 +1,11 @@
 #define GTEST_HAS_EXCEPTIONS 0
 
 #include <SDL/SDL.h>
+#include <SDL/SDL_error.h>
+#include <SDL/SDL_video.h>
+#include <stdio.h>
+#include <stdlib.h>
+
 #include "gtest/gtest.h"
 
 int main(int argc, char **argv) {
index b8b3cb4..195dcca 100644 (file)
--- a/init.cpp
+++ b/init.cpp
@@ -1,4 +1,7 @@
 #include <GL/glew.h>
+#include <assert.h>
+#include <stddef.h>
+#include <algorithm>
 #include <string>
 
 #include "init.h"
index 3301d46..3597e6c 100644 (file)
@@ -1,5 +1,5 @@
-#include <math.h>
 #include <GL/glew.h>
+#include <math.h>
 
 #include "lift_gamma_gain_effect.h"
 #include "util.h"
index 6835620..bd1ec16 100644 (file)
@@ -20,6 +20,9 @@
 // Also, gamma is a case where we would not want premultiplied alpha.
 // Thus, we have to divide away alpha first, and then re-multiply it back later.
 
+#include <GL/glew.h>
+#include <string>
+
 #include "effect.h"
 
 class LiftGammaGainEffect : public Effect {
index 3e63679..0b63063 100644 (file)
@@ -1,8 +1,10 @@
 // Unit tests for LiftGammaGainEffect.
 
-#include "test_util.h"
+#include "effect_chain.h"
 #include "gtest/gtest.h"
+#include "image_format.h"
 #include "lift_gamma_gain_effect.h"
+#include "test_util.h"
 
 TEST(LiftGammaGainEffectTest, DefaultIsNoop) {
        float data[] = {
index 03f3409..a33921b 100644 (file)
@@ -3,6 +3,8 @@
 
 // A simple horizontal mirroring.
 
+#include <string>
+
 #include "effect.h"
 
 class MirrorEffect : public Effect {
index 967c06b..4ef62fe 100644 (file)
@@ -4,6 +4,8 @@
 // Combine two images: a*x + b*y. If you set a within [0,1] and b=1-a,
 // you will get a fade; if not, you may get surprising results (consider alpha).
 
+#include <string>
+
 #include "effect.h"
 
 class MixEffect : public Effect {
index 096ecd5..babb766 100644 (file)
@@ -1,8 +1,13 @@
 // Unit tests for MixEffect.
 
-#include "test_util.h"
+#include <GL/glew.h>
+
+#include "effect_chain.h"
 #include "gtest/gtest.h"
+#include "image_format.h"
+#include "input.h"
 #include "mix_effect.h"
+#include "test_util.h"
 
 TEST(MixEffectTest, FiftyFiftyMix) {
        float data_a[] = {
index 4c6d37a..66ff31b 100644 (file)
@@ -10,6 +10,8 @@
 //
 // The first input is the bottom, and the second is the top.
 
+#include <string>
+
 #include "effect.h"
 
 class OverlayEffect : public Effect {
index d0867a1..5d86ec2 100644 (file)
@@ -1,8 +1,13 @@
 // Unit tests for OverlayEffect.
 
-#include "test_util.h"
+#include <GL/glew.h>
+
+#include "effect_chain.h"
 #include "gtest/gtest.h"
+#include "image_format.h"
+#include "input.h"
 #include "overlay_effect.h"
+#include "test_util.h"
 
 TEST(OverlayEffectTest, TopDominatesBottomWhenNoAlpha) {
        float data_a[] = {
index f8d30d4..f7f9707 100644 (file)
@@ -1,5 +1,5 @@
-#include <math.h>
 #include <GL/glew.h>
+#include <assert.h>
 
 #include "padding_effect.h"
 #include "util.h"
index 9e206ce..a836ba2 100644 (file)
@@ -12,6 +12,9 @@
 // You may not change it after calling finalize(), since that could change the
 // graph (need_linear_light() etc. depend on the border color you choose).
 
+#include <GL/glew.h>
+#include <string>
+
 #include "effect.h"
 
 class PaddingEffect : public Effect {
index e3f6f41..f86df3a 100644 (file)
@@ -1,9 +1,14 @@
 // Unit tests for AlphaMultiplicationEffect.
 
-#include "test_util.h"
+#include <stddef.h>
+
+#include "effect_chain.h"
 #include "flat_input.h"
-#include "padding_effect.h"
 #include "gtest/gtest.h"
+#include "image_format.h"
+#include "padding_effect.h"
+#include "test_util.h"
+#include "util.h"
 
 TEST(PaddingEffectTest, SimpleCenter) {
        float data[2 * 2] = {
index ec4f67e..264e5c1 100644 (file)
@@ -1,12 +1,15 @@
 // Three-lobed Lanczos, the most common choice.
 #define LANCZOS_RADIUS 3.0
 
-#include <math.h>
-#include <assert.h>
 #include <GL/glew.h>
+#include <assert.h>
+#include <limits.h>
+#include <math.h>
+#include <stdio.h>
+#include <algorithm>
 
-#include "resample_effect.h"
 #include "effect_chain.h"
+#include "resample_effect.h"
 #include "util.h"
 
 namespace {
index b38728f..b66ca6f 100644 (file)
 // which is what the user is intended to use, instantiates two copies of
 // SingleResamplePassEffect behind the scenes).
 
+#include <GL/glew.h>
+#include <assert.h>
+#include <stddef.h>
+#include <string>
+
 #include "effect.h"
 
+class EffectChain;
+class Node;
 class SingleResamplePassEffect;
 
 class ResampleEffect : public Effect {
index 462f5c9..a9c6923 100644 (file)
@@ -1,9 +1,14 @@
 // Unit tests for ResampleEffect.
 
-#include "test_util.h"
+#include <math.h>
+#include <iomanip>
+
+#include "effect_chain.h"
+#include "flat_input.h"
 #include "gtest/gtest.h"
+#include "image_format.h"
 #include "resample_effect.h"
-#include "flat_input.h"
+#include "test_util.h"
 
 namespace {
 
index 3cb40e3..8fbaba7 100644 (file)
@@ -5,6 +5,8 @@
 // (set by the two integer parameters "width" and "height").
 // Mostly useful as part of other algorithms.
 
+#include <string>
+
 #include "effect.h"
 
 class ResizeEffect : public Effect {
index 3f24a42..6aafb74 100644 (file)
@@ -1,5 +1,3 @@
-#include <math.h>
-#include <assert.h>
 #include <GL/glew.h>
 
 #include "sandbox_effect.h"
index 4443ff7..072bc2b 100644 (file)
@@ -8,6 +8,9 @@
 // throwaway code. When you're happy, you can do a bit of search and replace
 // to give it a proper name and its own place in the build system.
 
+#include <GL/glew.h>
+#include <string>
+
 #include "effect.h"
 
 class SandboxEffect : public Effect {
index 27b1d00..710dcfd 100644 (file)
@@ -7,6 +7,8 @@
 // (saturation=1). Extrapolating that curve further (ie., saturation > 1)
 // gives us increased saturation if so desired.
 
+#include <string>
+
 #include "effect.h"
 
 class SaturationEffect : public Effect {
index 6712197..dadb964 100644 (file)
@@ -1,8 +1,12 @@
 // Unit tests for SaturationEffect.
 
-#include "test_util.h"
+#include <GL/glew.h>
+
+#include "effect_chain.h"
 #include "gtest/gtest.h"
+#include "image_format.h"
 #include "saturation_effect.h"
+#include "test_util.h"
 
 TEST(SaturationEffectTest, SaturationOneIsPassThrough) {
        float data[] = {
index b89b3ff..05b61af 100644 (file)
@@ -1,12 +1,17 @@
-#include "init.h"
-#include "test_util.h"
+#include <assert.h>
+#include <gtest/gtest-message.h>
+#include <math.h>
+#include <stdio.h>
+#include <algorithm>
+#include <ostream>
+
 #include "flat_input.h"
 #include "gtest/gtest.h"
+#include "init.h"
+#include "test_util.h"
+#include "util.h"
 
-#include <stdio.h>
-#include <math.h>
-
-#include <algorithm>
+class Input;
 
 namespace {
 
index 9a7d001..1f48ae1 100644 (file)
@@ -1,7 +1,11 @@
 #ifndef _TEST_UTIL_H
 #define _TEST_UTIL_H 1
 
+#include <GL/glew.h>
 #include "effect_chain.h"
+#include "image_format.h"
+
+class Input;
 
 class EffectChainTester {
 public:
index 0ca22ee..7c28925 100644 (file)
@@ -1,10 +1,10 @@
-#include <math.h>
 #include <assert.h>
+#include <vector>
 
-#include "unsharp_mask_effect.h"
 #include "blur_effect.h"
-#include "mix_effect.h"
 #include "effect_chain.h"
+#include "mix_effect.h"
+#include "unsharp_mask_effect.h"
 #include "util.h"
 
 UnsharpMaskEffect::UnsharpMaskEffect()
index 45a04f0..b0c9377 100644 (file)
 // See DeconvolutionSharpenEffect for a different, possibly better
 // sharpening algorithm.
 
+#include <GL/glew.h>
+#include <assert.h>
+#include <string>
+
 #include "effect.h"
 
 class BlurEffect;
+class EffectChain;
 class MixEffect;
+class Node;
 
 class UnsharpMaskEffect : public Effect {
 public:
index a957ef1..8dbefc2 100644 (file)
@@ -2,8 +2,10 @@
 
 #include <math.h>
 
-#include "test_util.h"
+#include "effect_chain.h"
 #include "gtest/gtest.h"
+#include "image_format.h"
+#include "test_util.h"
 #include "unsharp_mask_effect.h"
 
 TEST(UnsharpMaskEffectTest, NoAmountDoesNothing) {
index 4e6a31c..a1bc2fd 100644 (file)
--- a/util.cpp
+++ b/util.cpp
@@ -1,11 +1,13 @@
-#include <stdio.h>
+#include <GL/glew.h>
+#include <assert.h>
 #include <math.h>
+#include <stdio.h>
+#include <stdlib.h>
 #include <string.h>
-#include <assert.h>
-#include <GL/glew.h>
+#include <Eigen/Core>
 
-#include "util.h"
 #include "init.h"
+#include "util.h"
 
 extern std::string *movit_data_directory;
 
diff --git a/util.h b/util.h
index 8917dd3..5839e5b 100644 (file)
--- a/util.h
+++ b/util.h
@@ -3,13 +3,11 @@
 
 // Various utilities.
 
+#include <GL/glew.h>
 #include <stdio.h>
 #include <stdlib.h>
-
-#include <string>
 #include <Eigen/Core>
-
-#include <GL/glew.h>
+#include <string>
 
 #define BUFFER_OFFSET(i) ((char *)NULL + (i))
 
index 83aaa10..bbd61da 100644 (file)
@@ -1,5 +1,5 @@
-#include <math.h>
 #include <GL/glew.h>
+#include <math.h>
 
 #include "vignette_effect.h"
 #include "util.h"
index a394703..d13bf85 100644 (file)
@@ -4,6 +4,9 @@
 // A circular vignette, falling off as cos² of the distance from the center
 // (the classic formula for approximating a real lens).
 
+#include <GL/glew.h>
+#include <string>
+
 #include "effect.h"
 
 class VignetteEffect : public Effect {
index b00f0ca..1583922 100644 (file)
@@ -1,12 +1,11 @@
-#include <math.h>
-#include <assert.h>
-#include <GL/glew.h>
-
+#include <Eigen/Core>
 #include <Eigen/LU>
+#include <GL/glew.h>
+#include <assert.h>
 
-#include "white_balance_effect.h"
-#include "util.h"
 #include "d65.h"
+#include "util.h"
+#include "white_balance_effect.h"
 
 using namespace Eigen;
 
index 2f8f38e..c4e46a4 100644 (file)
@@ -3,6 +3,9 @@
 
 // Color correction in LMS color space.
 
+#include <GL/glew.h>
+#include <string>
+
 #include "effect.h"
 
 class WhiteBalanceEffect : public Effect {
index 733b5bc..4cd2ae0 100644 (file)
@@ -1,7 +1,11 @@
 // Unit tests for WhiteBalanceEffect.
 
-#include "test_util.h"
+#include <iomanip>
+
+#include "effect_chain.h"
 #include "gtest/gtest.h"
+#include "image_format.h"
+#include "test_util.h"
 #include "white_balance_effect.h"
 
 TEST(WhiteBalanceEffectTest, GrayNeutralDoesNothing) {
index 5742e45..1c73136 100644 (file)
@@ -1,5 +1,5 @@
-#include <math.h>
 #include <GL/glew.h>
+#include <math.h>
 
 #include "widgets.h"
 #include "util.h"
index 098b574..3e691d8 100644 (file)
@@ -1,11 +1,12 @@
-#include <string.h>
-#include <assert.h>
-#include <GL/glew.h>
-
+#include <Eigen/Core>
 #include <Eigen/LU>
+#include <GL/glew.h>
+#include <assert.h>
+#include <stdio.h>
+#include <string.h>
 
-#include "ycbcr_input.h"
 #include "util.h"
+#include "ycbcr_input.h"
 
 using namespace Eigen;
 
index 492230f..2220f5d 100644 (file)
@@ -5,6 +5,12 @@
 // imprecisely, called “YUV”), which is typically what you get from a video decoder.
 // It upsamples planes as needed, using the default linear upsampling OpenGL gives you.
 
+#include <GL/glew.h>
+#include <assert.h>
+#include <string>
+
+#include "effect.h"
+#include "image_format.h"
 #include "input.h"
 
 struct YCbCrFormat {
index 5bd052b..15d7639 100644 (file)
@@ -1,8 +1,11 @@
 // Unit tests for YCbCrInput.
 // FIXME: This class really ought to support mipmaps.
 
-#include "test_util.h"
+#include <stddef.h>
+
+#include "effect_chain.h"
 #include "gtest/gtest.h"
+#include "test_util.h"
 #include "ycbcr_input.h"
 
 TEST(YCbCrInput, Simple444) {