From: Steinar H. Gunderson <sgunderson@bigfoot.com>
Date: Sun, 1 Nov 2015 19:55:51 +0000 (+0100)
Subject: Re-run IWYU, again with lots of manual cleanup.
X-Git-Tag: 1.0.0~179
X-Git-Url: https://git.sesse.net/?p=nageru;a=commitdiff_plain;h=d7bba4abb3d56893399578f0540b9ded0a28380f

Re-run IWYU, again with lots of manual cleanup.
---

diff --git a/flags.cpp b/flags.cpp
index e0c638b..56ca799 100644
--- a/flags.cpp
+++ b/flags.cpp
@@ -1,9 +1,8 @@
-#include <stdio.h>
+#include "flags.h"
+
 #include <getopt.h>
+#include <stdio.h>
 #include <stdlib.h>
-#include <unistd.h>
-
-#include "flags.h"
 
 Flags global_flags;
 
diff --git a/glwidget.cpp b/glwidget.cpp
index da5be6f..a309240 100644
--- a/glwidget.cpp
+++ b/glwidget.cpp
@@ -9,20 +9,21 @@
 #include <QSurfaceFormat>
 
 #include "glwidget.h"
-#include "mainwindow.h"
 
 #include <movit/resource_pool.h>
 #include <stdio.h>
+#include <functional>
 #include <mutex>
 
 #include "context.h"
+#include "effect_chain.h"
 #include "flags.h"
+#include "mainwindow.h"
 #include "mixer.h"
+#include "qnamespace.h"
 #include "ref_counted_gl_sync.h"
-#include "vumeter.h"
 
-class MainWindow;
-class QSurface;
+class QMouseEvent;
 class QWidget;
 
 #undef Success
diff --git a/glwidget.h b/glwidget.h
index 3641f66..25a8ad7 100644
--- a/glwidget.h
+++ b/glwidget.h
@@ -3,9 +3,14 @@
 
 #include <epoxy/gl.h>
 #include <QGLWidget>
+#include <string>
+#include <vector>
 
 #include "mixer.h"
+#include "qgl.h"
+#include "qobjectdefs.h"
 
+class QMouseEvent;
 class QWidget;
 
 namespace movit {
diff --git a/h264encode.cpp b/h264encode.cpp
index 74b5191..51ad517 100644
--- a/h264encode.cpp
+++ b/h264encode.cpp
@@ -7,11 +7,11 @@
 #include <assert.h>
 #include <epoxy/egl.h>
 #include <libavcodec/avcodec.h>
-#include <libavformat/avio.h>
-#include <libavutil/mathematics.h>
+#include <libavutil/channel_layout.h>
+#include <libavutil/frame.h>
 #include <libavutil/rational.h>
+#include <libavutil/samplefmt.h>
 #include <libdrm/drm_fourcc.h>
-#include <stdint.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -20,10 +20,13 @@
 #include <va/va_enc_h264.h>
 #include <va/va_x11.h>
 #include <condition_variable>
+#include <cstdint>
+#include <memory>
 #include <mutex>
 #include <queue>
 #include <string>
 #include <thread>
+#include <utility>
 
 #include "context.h"
 #include "httpd.h"
diff --git a/h264encode.h b/h264encode.h
index 7a10d92..2bbbed5 100644
--- a/h264encode.h
+++ b/h264encode.h
@@ -27,9 +27,12 @@
 #define _H264ENCODE_H
 
 extern "C" {
+#include <libavcodec/avcodec.h>
 #include <libavformat/avformat.h>
 }
 #include <epoxy/gl.h>
+#include <stdbool.h>
+#include <stdint.h>
 #include <atomic>
 #include <condition_variable>
 #include <map>
diff --git a/httpd.cpp b/httpd.cpp
index f266c63..db3cb61 100644
--- a/httpd.cpp
+++ b/httpd.cpp
@@ -1,14 +1,26 @@
-#include <string.h>
-#include <microhttpd.h>
 #include <assert.h>
+#include <microhttpd.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
 
 extern "C" {
 #include <libavcodec/avcodec.h>
+#include <libavutil/channel_layout.h>
+#include <libavutil/mathematics.h>
+#include <libavutil/mem.h>
+#include <libavutil/pixfmt.h>
+#include <libavutil/rational.h>
+#include <libavutil/samplefmt.h>
 }
 
 #include "httpd.h"
+
 #include "timebase.h"
 
+struct MHD_Connection;
+struct MHD_Response;
+
 using namespace std;
 
 HTTPD::HTTPD(const char *output_filename, int width, int height)
diff --git a/httpd.h b/httpd.h
index 524c8d6..6c746c0 100644
--- a/httpd.h
+++ b/httpd.h
@@ -9,14 +9,22 @@
 // way, namely having one ffmpeg mux per output.
 
 #include <microhttpd.h>
+#include <stddef.h>
+#include <stdint.h>
+#include <sys/types.h>
+#include <condition_variable>
 #include <deque>
-#include <string>
+#include <memory>
 #include <mutex>
-#include <condition_variable>
+#include <string>
 #include <vector>
 
+struct MHD_Connection;
+
 extern "C" {
+#include <libavcodec/avcodec.h>
 #include <libavformat/avformat.h>
+#include <libavformat/avio.h>
 }
 
 class HTTPD {
diff --git a/lrameter.cpp b/lrameter.cpp
index cd87a6c..6e9fed4 100644
--- a/lrameter.cpp
+++ b/lrameter.cpp
@@ -1,8 +1,12 @@
-#include <QPainter>
-
 #include "lrameter.h"
+
+#include <QPainter>
+#include <QPalette>
+#include <QPen>
 #include "vu_common.h"
 
+class QPaintEvent;
+
 using namespace std;
 
 LRAMeter::LRAMeter(QWidget *parent)
diff --git a/lrameter.h b/lrameter.h
index 50497be..bcd8377 100644
--- a/lrameter.h
+++ b/lrameter.h
@@ -3,10 +3,10 @@
 #ifndef LRAMETER_H
 #define LRAMETER_H
 
-#include <QWidget>
+#include <math.h>
 #include <QLabel>
 #include <QPaintEvent>
-
+#include <QWidget>
 #include <mutex>
 
 class LRAMeter : public QWidget
diff --git a/main.cpp b/main.cpp
index 5917c7e..fd41e0b 100644
--- a/main.cpp
+++ b/main.cpp
@@ -1,3 +1,8 @@
+extern "C" {
+#include <libavformat/avformat.h>
+}
+#include <stdbool.h>
+#include <stdio.h>
 #include <stdlib.h>
 #include <epoxy/gl.h>
 
diff --git a/mainwindow.cpp b/mainwindow.cpp
index ecdb131..3f0098b 100644
--- a/mainwindow.cpp
+++ b/mainwindow.cpp
@@ -1,16 +1,28 @@
 #include "mainwindow.h"
-#include <thread>
-#include <vector>
+
+#include <math.h>
+#include <stdio.h>
+#include <algorithm>
 #include <string>
-#include <QSignalMapper>
+#include <vector>
+#include <QBoxLayout>
+#include <QKeySequence>
+#include <QLabel>
 #include <QMetaType>
-#include <QShortcut>
+#include <QPushButton>
 #include <QResizeEvent>
+#include <QShortcut>
+#include <QSignalMapper>
+#include <QSize>
+#include <QString>
 
-#include "context.h"
+#include "glwidget.h"
+#include "lrameter.h"
 #include "mixer.h"
-
 #include "ui_mainwindow.h"
+#include "vumeter.h"
+
+class QResizeEvent;
 
 using namespace std;
 
diff --git a/mainwindow.h b/mainwindow.h
index 6f26668..ef0b04f 100644
--- a/mainwindow.h
+++ b/mainwindow.h
@@ -2,8 +2,10 @@
 #define MAINWINDOW_H
 
 #include <QMainWindow>
-#include <vector>
 #include <string>
+#include <vector>
+
+class QResizeEvent;
 
 namespace Ui {
 class MainWindow;
diff --git a/mixer.cpp b/mixer.cpp
index cf2ed83..981c0cb 100644
--- a/mixer.cpp
+++ b/mixer.cpp
@@ -7,27 +7,20 @@
 #include "mixer.h"
 
 #include <assert.h>
-#include <effect.h>
-#include <effect_chain.h>
-#include <effect_util.h>
 #include <epoxy/egl.h>
-#include <features.h>
-#include <image_format.h>
 #include <init.h>
-#include <overlay_effect.h>
-#include <padding_effect.h>
-#include <resample_effect.h>
-#include <resource_pool.h>
-#include <saturation_effect.h>
+#include <movit/effect_chain.h>
+#include <movit/effect_util.h>
+#include <movit/flat_input.h>
+#include <movit/image_format.h>
+#include <movit/resource_pool.h>
 #include <stdint.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <sys/time.h>
 #include <time.h>
 #include <util.h>
-#include <white_balance_effect.h>
-#include <ycbcr.h>
-#include <ycbcr_input.h>
+#include <algorithm>
 #include <cmath>
 #include <condition_variable>
 #include <cstddef>
@@ -35,6 +28,7 @@
 #include <mutex>
 #include <string>
 #include <thread>
+#include <utility>
 #include <vector>
 
 #include "bmusb/bmusb.h"
diff --git a/mixer.h b/mixer.h
index 82a3cca..bf53c24 100644
--- a/mixer.h
+++ b/mixer.h
@@ -5,20 +5,39 @@
 
 #include <epoxy/gl.h>
 #undef Success
+#include <stdbool.h>
+#include <stdint.h>
+
 #include <movit/effect_chain.h>
 #include <movit/flat_input.h>
+#include <condition_variable>
+#include <cstddef>
 #include <functional>
+#include <memory>
+#include <mutex>
+#include <string>
+#include <thread>
+#include <vector>
 
 #include "bmusb/bmusb.h"
+#include "ebu_r128_proc.h"
 #include "h264encode.h"
+#include "httpd.h"
 #include "pbo_frame_allocator.h"
 #include "ref_counted_frame.h"
 #include "ref_counted_gl_sync.h"
-#include "theme.h"
 #include "resampler.h"
+#include "theme.h"
 #include "timebase.h"
-#include "httpd.h"
-#include "ebu_r128_proc.h"
+
+class H264Encoder;
+class QSurface;
+namespace movit {
+class Effect;
+class EffectChain;
+class FlatInput;
+class ResourcePool;
+}  // namespace movit
 
 namespace movit {
 class YCbCrInput;
diff --git a/pbo_frame_allocator.cpp b/pbo_frame_allocator.cpp
index 204bb94..1c370b4 100644
--- a/pbo_frame_allocator.cpp
+++ b/pbo_frame_allocator.cpp
@@ -1,5 +1,6 @@
 #include "pbo_frame_allocator.h"
 
+#include <stdbool.h>
 #include <stdint.h>
 #include <stdio.h>
 #include <cstddef>
diff --git a/pbo_frame_allocator.h b/pbo_frame_allocator.h
index 1fb6a07..452c006 100644
--- a/pbo_frame_allocator.h
+++ b/pbo_frame_allocator.h
@@ -2,6 +2,8 @@
 #define _PBO_FRAME_ALLOCATOR 1
 
 #include <epoxy/gl.h>
+#include <stddef.h>
+#include <memory>
 #include <mutex>
 #include <queue>
 
diff --git a/ref_counted_frame.h b/ref_counted_frame.h
index 6d66a76..67787ef 100644
--- a/ref_counted_frame.h
+++ b/ref_counted_frame.h
@@ -7,6 +7,8 @@
 // Note that the important point isn't really the pointer to the Frame itself,
 // it's the resources it's representing that need to go back to the allocator.
 
+#include <memory>
+
 #include "bmusb/bmusb.h"
 
 void release_refcounted_frame(FrameAllocator::Frame *frame);
diff --git a/resampler.cpp b/resampler.cpp
index ddcc229..138f837 100644
--- a/resampler.cpp
+++ b/resampler.cpp
@@ -18,9 +18,10 @@
 
 #include "resampler.h"
 
+#include <math.h>
+#include <stddef.h>
 #include <stdio.h>
 #include <string.h>
-#include <math.h>
 #include <zita-resampler/vresampler.h>
 
 Resampler::Resampler(unsigned freq_in, unsigned freq_out, unsigned num_channels)
diff --git a/resampler.h b/resampler.h
index 85fea83..7d3b602 100644
--- a/resampler.h
+++ b/resampler.h
@@ -40,9 +40,10 @@
 
 #include <stdint.h>
 #include <stdlib.h>
+#include <sys/types.h>
 #include <zita-resampler/vresampler.h>
-
 #include <deque>
+#include <memory>
 
 class Resampler {
 public:
diff --git a/theme.cpp b/theme.cpp
index 2a84093..f32f637 100644
--- a/theme.cpp
+++ b/theme.cpp
@@ -1,20 +1,29 @@
-#include <stdio.h>
+#include "theme.h"
+
+#include <assert.h>
+#include <lauxlib.h>
 #include <lua.h>
 #include <lualib.h>
-#include <lauxlib.h>
-#include <new>
-#include <utility>
-
+#include <movit/effect.h>
 #include <movit/effect_chain.h>
-#include <movit/ycbcr_input.h>
-#include <movit/white_balance_effect.h>
-#include <movit/resample_effect.h>
-#include <movit/padding_effect.h>
+#include <movit/image_format.h>
+#include <movit/mix_effect.h>
 #include <movit/overlay_effect.h>
+#include <movit/padding_effect.h>
+#include <movit/resample_effect.h>
 #include <movit/resize_effect.h>
-#include <movit/mix_effect.h>
+#include <movit/white_balance_effect.h>
+#include <movit/ycbcr.h>
+#include <movit/ycbcr_input.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <cstddef>
+#include <new>
+#include <utility>
 
-#include "theme.h"
+namespace movit {
+class ResourcePool;
+}  // namespace movit
 
 #define WIDTH 1280  // FIXME
 #define HEIGHT 720  // FIXME
diff --git a/theme.h b/theme.h
index b17dbbd..2b75aa1 100644
--- a/theme.h
+++ b/theme.h
@@ -1,16 +1,24 @@
 #ifndef _THEME_H
 #define _THEME_H 1
 
-#include <stdio.h>
-#include <lua.h>
+#include <epoxy/gl.h>
 #include <lauxlib.h>
-
+#include <lua.h>
+#include <movit/effect_chain.h>
+#include <movit/ycbcr_input.h>
+#include <stdbool.h>
+#include <stdio.h>
 #include <functional>
 #include <mutex>
+#include <string>
 #include <utility>
+#include <vector>
 
-#include <movit/effect_chain.h>
-#include <movit/ycbcr_input.h>
+namespace movit {
+class ResourcePool;
+struct ImageFormat;
+struct YCbCrFormat;
+}  // namespace movit
 
 #define MAX_CARDS 16
 
diff --git a/vu_common.cpp b/vu_common.cpp
index 9fc3553..7b14f2f 100644
--- a/vu_common.cpp
+++ b/vu_common.cpp
@@ -1,7 +1,12 @@
 #include "vu_common.h"
+
 #include <math.h>
 #include <algorithm>
 
+#include <QBrush>
+#include <QColor>
+#include <QPainter>
+
 using namespace std;
 
 int lufs_to_pos(float level_lu, int height)
diff --git a/vumeter.h b/vumeter.h
index 720a12e..12afdc6 100644
--- a/vumeter.h
+++ b/vumeter.h
@@ -1,10 +1,10 @@
 #ifndef VUMETER_H
 #define VUMETER_H
 
-#include <QWidget>
+#include <math.h>
 #include <QLabel>
 #include <QPaintEvent>
-
+#include <QWidget>
 #include <mutex>
 
 class VUMeter : public QWidget