From d147175aaa3e5abfe271940c4518e9ef8dbbc932 Mon Sep 17 00:00:00 2001
From: "Steinar H. Gunderson" <sgunderson@bigfoot.com>
Date: Sun, 4 Oct 2015 22:49:25 +0200
Subject: [PATCH] Run IWYU (plus lots of manual fiddling).

---
 bmusb.cpp               | 26 +++++++++---------
 bmusb.h                 |  3 +++
 context.cpp             |  9 ++++---
 glwidget.cpp            | 30 ++++++++++++---------
 glwidget.h              |  3 +++
 h264encode.cpp          | 42 ++++++++++++++++-------------
 h264encode.h            | 11 +++++---
 main.cpp                | 10 ++++---
 mixer.cpp               | 59 +++++++++++++++++++----------------------
 mixer.h                 |  4 +++
 pbo_frame_allocator.cpp |  5 ++++
 pbo_frame_allocator.h   |  2 +-
 window.cpp              |  7 ++---
 13 files changed, 122 insertions(+), 89 deletions(-)

diff --git a/bmusb.cpp b/bmusb.cpp
index 207d632..65c3b1c 100644
--- a/bmusb.cpp
+++ b/bmusb.cpp
@@ -6,29 +6,31 @@
 // 576p60/720p60/1080i60 works, 1080p60 does not work (firmware limitation)
 // Audio comes out as 8-channel 24-bit raw audio.
 
+#include <assert.h>
+#include <errno.h>
+#include <libusb.h>
+#include <netinet/in.h>
+#include <sched.h>
+#include <stdint.h>
 #include <stdio.h>
 #include <stdlib.h>
-#include <libusb.h>
-#include <arpa/inet.h>
-#include <unistd.h>
 #include <string.h>
-#include <fcntl.h>
-#include <stdint.h>
-#include <assert.h>
 #ifdef __SSE2__
 #include <immintrin.h>
 #endif
+#include "bmusb.h"
+
 #include <algorithm>
+#include <atomic>
+#include <condition_variable>
+#include <cstddef>
+#include <cstdint>
+#include <deque>
 #include <functional>
 #include <memory>
-#include <deque>
-#include <utility>
 #include <mutex>
-#include <condition_variable>
-#include <thread>
 #include <stack>
-#include <atomic>
-#include "bmusb.h"
+#include <thread>
 
 using namespace std;
 using namespace std::placeholders;
diff --git a/bmusb.h b/bmusb.h
index c881748..301350f 100644
--- a/bmusb.h
+++ b/bmusb.h
@@ -8,6 +8,9 @@
 #include <functional>
 #include <mutex>
 #include <thread>
+#include <vector>
+
+struct libusb_transfer;
 
 // An interface for frame allocators; if you do not specify one
 // (using set_video_frame_allocator), a default one that pre-allocates
diff --git a/context.cpp b/context.cpp
index 946d40d..4ae85bd 100644
--- a/context.cpp
+++ b/context.cpp
@@ -1,10 +1,11 @@
 #include <stdio.h>
-#include <stdlib.h>
 
-#include <QOpenGLContext>
+#include <QGL>
 #include <QOffscreenSurface>
-#include <QWindow>
-#include <QGLWidget>
+#include <QOpenGLContext>
+#include <QSurfaceFormat>
+
+class QSurface;
 
 QGLWidget *global_share_widget = nullptr;
 
diff --git a/glwidget.cpp b/glwidget.cpp
index 6e216a6..319592e 100644
--- a/glwidget.cpp
+++ b/glwidget.cpp
@@ -1,21 +1,27 @@
-#include <qmetatype.h>
-#include <qdatastream.h>
-#include <qtextstream.h>
-#include <qcursor.h>
-#include <qcoreevent.h>
+#include <qmetatype.h>  // Needs to come before egl.h.
+#include <qdatastream.h>  // Needs to come before egl.h.
+#include <qtextstream.h>  // Needs to come before egl.h.
+#include <qcursor.h>  // Needs to come before egl.h.
+#include <qcoreevent.h>  // Needs to come before egl.h.
 #include <epoxy/gl.h>
 #include <epoxy/egl.h>
-#include "context.h"
+#include <QSurfaceFormat>
+
 #include "glwidget.h"
-#include "mixer.h"
-#include <QCoreApplication>
-#include <QGuiApplication>
-#include <QThread>
-#include <math.h>
-#include <thread>
+
 #include <movit/resource_pool.h>
+#include <stdio.h>
+
+#include "context.h"
+#include "mixer.h"
+#include "ref_counted_gl_sync.h"
+
+class QSurface;
+class QWidget;
+
 #undef Success
 #include <movit/util.h>
+#include <string>
 
 GLWidget::GLWidget(QWidget *parent)
     : QGLWidget(parent, global_share_widget),
diff --git a/glwidget.h b/glwidget.h
index bb62940..1f6b6e7 100644
--- a/glwidget.h
+++ b/glwidget.h
@@ -1,8 +1,11 @@
 #ifndef GLWIDGET_H
 #define GLWIDGET_H
 
+#include <epoxy/gl.h>
 #include <QGLWidget>
 
+class QWidget;
+
 namespace movit {
 class ResourcePool;
 }
diff --git a/h264encode.cpp b/h264encode.cpp
index 83de3bd..db0ba93 100644
--- a/h264encode.cpp
+++ b/h264encode.cpp
@@ -1,28 +1,34 @@
 //#include "sysdeps.h"
+#include "h264encode.h"
+
+#include <EGL/eglplatform.h>
+#include <X11/X.h>
+#include <X11/Xlib.h>
+#include <assert.h>
+#include <epoxy/egl.h>
+#include <libavcodec/avcodec.h>
+#include <libavformat/avio.h>
+#include <libavutil/mathematics.h>
+#include <libavutil/rational.h>
+#include <libdrm/drm_fourcc.h>
+#include <stdint.h>
 #include <stdio.h>
-#include <string.h>
 #include <stdlib.h>
-#include <getopt.h>
-#include <unistd.h>
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <sys/time.h>
-#include <sys/mman.h>
-#include <fcntl.h>
-#include <assert.h>
-#include <pthread.h>
-#include <errno.h>
-#include <math.h>
+#include <string.h>
 #include <va/va.h>
-#include <va/va_x11.h>
-#include <va/va_enc_h264.h>
 #include <va/va_drmcommon.h>
-#include <libdrm/drm_fourcc.h>
-#include <thread>
+#include <va/va_enc_h264.h>
+#include <va/va_x11.h>
+#include <condition_variable>
 #include <mutex>
 #include <queue>
-#include <condition_variable>
-#include "h264encode.h"
+#include <string>
+#include <thread>
+
+#include "context.h"
+
+class QOpenGLContext;
+class QSurface;
 
 #define CHECK_VASTATUS(va_status, func)                                 \
     if (va_status != VA_STATUS_SUCCESS) {                               \
diff --git a/h264encode.h b/h264encode.h
index b14ca6f..b9effa7 100644
--- a/h264encode.h
+++ b/h264encode.h
@@ -29,18 +29,23 @@
 extern "C" {
 #include <libavformat/avformat.h>
 }
-#include <epoxy/egl.h>
+#include <epoxy/gl.h>
 #include <atomic>
+#include <condition_variable>
 #include <map>
 #include <memory>
 #include <mutex>
+#include <queue>
 #include <thread>
-#include <condition_variable>
+#include <vector>
 
-#include "pbo_frame_allocator.h"
+#include "bmusb.h"
 #include "context.h"
+#include "pbo_frame_allocator.h"
 #include "ref_counted_gl_sync.h"
 
+class QSurface;
+
 #define SURFACE_NUM 16 /* 16 surfaces for source YUV */
 
 class H264Encoder {
diff --git a/main.cpp b/main.cpp
index 9736395..f92b12e 100644
--- a/main.cpp
+++ b/main.cpp
@@ -1,13 +1,15 @@
+#include <stdlib.h>
 #include <epoxy/gl.h>
+
 #include <QApplication>
-#include <QDesktopWidget>
+#include <QCoreApplication>
+#include <QGL>
+#include <QSize>
 #include <QSurfaceFormat>
-#include <QtGui/QOpenGLContext>
-#include <QGLFormat>
 
+#include "context.h"
 #include "mainwindow.h"
 #include "mixer.h"
-#include "context.h"
 
 int main(int argc, char *argv[])
 {
diff --git a/mixer.cpp b/mixer.cpp
index 0c36c48..fec83ff 100644
--- a/mixer.cpp
+++ b/mixer.cpp
@@ -5,54 +5,49 @@
 #define WIDTH 1280
 #define HEIGHT 720
 
-#include <epoxy/gl.h>
-#include <epoxy/egl.h>
-
 #undef Success
 
-#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 <sys/time.h>
-#include <time.h>
-#include <mutex>
-#include <queue>
-#include <condition_variable>
+#include "mixer.h"
 
-#include <diffusion_effect.h>
+#include <assert.h>
 #include <effect.h>
 #include <effect_chain.h>
-#include <flat_input.h>
+#include <effect_util.h>
+#include <epoxy/egl.h>
+#include <features.h>
 #include <image_format.h>
 #include <init.h>
-#include <lift_gamma_gain_effect.h>
-#include <saturation_effect.h>
-#include <util.h>
-#include <ycbcr_input.h>
-#include <vignette_effect.h>
-#include <resample_effect.h>
-#include <resize_effect.h>
 #include <overlay_effect.h>
 #include <padding_effect.h>
+#include <resample_effect.h>
+#include <resource_pool.h>
+#include <saturation_effect.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 <resource_pool.h>
-#include <effect_util.h>
-
-#include <EGL/egl.h>
+#include <ycbcr_input.h>
+#include <cmath>
+#include <condition_variable>
+#include <cstddef>
+#include <memory>
+#include <mutex>
+#include <string>
+#include <thread>
+#include <vector>
 
-#include "h264encode.h"
-#include "context.h"
 #include "bmusb.h"
+#include "context.h"
+#include "h264encode.h"
 #include "pbo_frame_allocator.h"
-#include "mixer.h"
 #include "ref_counted_gl_sync.h"
 
+class QOpenGLContext;
+
 using namespace movit;
 using namespace std;
 using namespace std::placeholders;
diff --git a/mixer.h b/mixer.h
index 2c6e51c..70953bd 100644
--- a/mixer.h
+++ b/mixer.h
@@ -1,9 +1,13 @@
 #ifndef _MIXER_H
 #define _MIXER_H 1
 
+#include <epoxy/gl.h>
+#include <functional>
+
 #include "ref_counted_gl_sync.h"
 
 class QSurface;
+
 void start_mixer(QSurface *surface, QSurface *surface2, QSurface *surface3, QSurface *surface4);
 void mixer_quit();
 
diff --git a/pbo_frame_allocator.cpp b/pbo_frame_allocator.cpp
index 59013bf..1018403 100644
--- a/pbo_frame_allocator.cpp
+++ b/pbo_frame_allocator.cpp
@@ -1,4 +1,9 @@
 #include "pbo_frame_allocator.h"
+
+#include <stdint.h>
+#include <stdio.h>
+#include <cstddef>
+
 #include "util.h"
 
 using namespace std;
diff --git a/pbo_frame_allocator.h b/pbo_frame_allocator.h
index 1155d20..825d01f 100644
--- a/pbo_frame_allocator.h
+++ b/pbo_frame_allocator.h
@@ -1,9 +1,9 @@
 #ifndef _PBO_FRAME_ALLOCATOR 
 #define _PBO_FRAME_ALLOCATOR 1
 
+#include <epoxy/gl.h>
 #include <mutex>
 #include <queue>
-#include <epoxy/gl.h>
 
 #include "bmusb.h"
 
diff --git a/window.cpp b/window.cpp
index b8eced7..6c67f78 100644
--- a/window.cpp
+++ b/window.cpp
@@ -1,8 +1,9 @@
-#include "glwidget.h"
 #include "window.h"
-#include "mainwindow.h"
-#include <QApplication>
+
 #include <QBoxLayout>
+#include <QString>
+
+#include "glwidget.h"
 
 Window::Window(MainWindow *mw)
 	: main_window(mw)
-- 
2.39.2