include: avoid including vlc_subpicture.h when not needed

We should not rebuild a lot of Qt files when changing the subpictures.

It's still included in vlc_codec.h so all encoders/decoders are still
impacted by such changes.

include/vlc_decoder.h

@@ -29,9 +29,10 @@
  * @{
  */
 
-#include <vlc_subpicture.h>
+#include <vlc_tick.h>
 
 typedef struct vlc_input_decoder_t vlc_input_decoder_t;
+typedef struct vlc_spu_highlight_t vlc_spu_highlight_t;
 
 /**
  * This defines an opaque input resource handler.

include/vlc_spu.h

@@ -25,7 +25,8 @@
 #ifndef VLC_SPU_H
 #define VLC_SPU_H 1
 
-#include <vlc_subpicture.h>
+#include <vlc_common.h>
+#include <vlc_tick.h>
 
 #ifdef __cplusplus
 extern "C" {

include/vlc_vout.h

@@ -27,7 +27,6 @@
 
 #include <vlc_es.h>
 #include <vlc_picture.h>
-#include <vlc_subpicture.h>
 
 /**
  * \defgroup output Output

include/vlc_vout_display.h

@@ -25,7 +25,6 @@
 
 #include <vlc_es.h>
 #include <vlc_picture.h>
-#include <vlc_subpicture.h>
 #include <vlc_mouse.h>
 #include <vlc_vout.h>
 #include <vlc_window.h>

lib/media_player.c

@@ -35,6 +35,7 @@
 #include <vlc_demux.h>
 #include <vlc_vout.h>
 #include <vlc_aout.h>
+#include <vlc_subpicture.h>
 #include <vlc_actions.h>
 #include <vlc_modules.h>
 

modules/access/bluray.c

@@ -51,6 +51,7 @@
 #include <vlc_url.h>                        /* vlc_path2uri */
 #include <vlc_iso_lang.h>
 #include <vlc_fs.h>
+#include <vlc_subpicture.h> // vlc_spu_highlight_t
 
 #include "../demux/mpeg/timestamps.h"
 #include "../demux/timestamps_filter.h"

modules/access/dvdnav.c

@@ -54,6 +54,7 @@
 #include <vlc_mouse.h>
 #include <vlc_dialog.h>
 #include <vlc_iso_lang.h>
+#include <vlc_subpicture.h> // vlc_spu_highlight_t
 
 #include <dvdnav/dvdnav.h>
 /* Expose without patching headers */

modules/demux/mkv/events.cpp

@@ -27,6 +27,7 @@
 
 #include <vlc_actions.h>
 #include <vlc_threads.h>
+#include <vlc_subpicture.h> // vlc_spu_highlight_t
 
 #include <algorithm>
 

modules/lua/libs/osd.c

@@ -33,6 +33,7 @@
 
 #include <vlc_vout.h>
 #include <vlc_vout_osd.h>
+#include <vlc_subpicture.h>
 
 #include "../vlc.h"
 #include "../libs.h"

modules/video_output/android/display.c

@@ -32,6 +32,7 @@
 #include <vlc_threads.h>
 #include <vlc_plugin.h>
 #include <vlc_vout_display.h>
+#include <vlc_subpicture.h>
 
 #include <vlc_vector.h>
 #include <vlc_opengl.h>

modules/video_output/libplacebo/display.c

@@ -31,6 +31,7 @@
 #include <vlc_plugin.h>
 #include <vlc_vout_display.h>
 #include <vlc_fs.h>
+#include <vlc_subpicture.h>
 
 #include "utils.h"
 #include "instance.h"

modules/video_output/xcb/render.c

@@ -39,6 +39,7 @@
 #include <vlc_fs.h>
 #include <vlc_plugin.h>
 #include <vlc_vout_display.h>
+#include <vlc_subpicture.h>
 
 #include "pictures.h"
 #include "events.h"

src/input/es_out_timeshift.c

@@ -43,6 +43,7 @@
 #include <vlc_mouse.h>
 #include <vlc_es_out.h>
 #include <vlc_block.h>
+#include <vlc_subpicture.h> // vlc_spu_highlight_t
 #include "input_internal.h"
 #ifdef _WIN32
 #  include <vlc_charset.h> // FromWide

src/video_output/video_text.c

@@ -30,6 +30,7 @@
 #include <vlc_threads.h>
 #include <vlc_vout.h>
 #include <vlc_vout_osd.h>
+#include <vlc_subpicture.h>
 
 typedef struct {
     int  position;

src/video_output/vout_internal.h

@@ -28,6 +28,7 @@
 #include <vlc_clock.h>
 
 typedef struct input_thread_t input_thread_t;
+typedef struct vlc_spu_highlight_t vlc_spu_highlight_t;
 
 /**
  * Vout configuration

src/video_output/vout_intf.c

@@ -42,6 +42,7 @@
 #include <vlc_strings.h>
 #include <vlc_charset.h>
 #include <vlc_spu.h>
+#include <vlc_subpicture.h>
 #include "vout_internal.h"
 #include "snapshot.h"