[vlc-devel] [PATCH 3/5] demux/mp4: move some stuff in a header file
Rémi Denis-Courmont
remi at remlab.net
Wed Apr 11 12:00:58 CEST 2012
On Wed, 11 Apr 2012 11:40:04 +0200, Frédéric Yhuel <fyhuel at viotech.net>
wrote:
> I need to include mp4_track_t definition in my Smooth Streaming module
demux_sys_t is intrinsically an internal structure of the MP4 demux. I
don't really see why you'd have to export it.
> ---
> modules/demux/mp4/Modules.am | 1 +
> modules/demux/mp4/mp4.c | 109
+-----------------------------------
> modules/demux/mp4/mp4.h | 130
> ++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 132 insertions(+), 108 deletions(-)
> create mode 100644 modules/demux/mp4/mp4.h
>
> diff --git a/modules/demux/mp4/Modules.am b/modules/demux/mp4/Modules.am
> index e9fa0b7..c4d7167 100644
> --- a/modules/demux/mp4/Modules.am
> +++ b/modules/demux/mp4/Modules.am
> @@ -1,5 +1,6 @@
> SOURCES_mp4 = \
> mp4.c \
> + mp4.h \
> libmp4.c \
> libmp4.h \
> id3genres.h \
> diff --git a/modules/demux/mp4/mp4.c b/modules/demux/mp4/mp4.c
> index 8faba8e..cd63051 100644
> --- a/modules/demux/mp4/mp4.c
> +++ b/modules/demux/mp4/mp4.c
> @@ -28,15 +28,7 @@
> # include "config.h"
> #endif
>
> -#include <vlc_common.h>
> -#include <vlc_plugin.h>
> -
> -#include <vlc_demux.h>
> -#include <vlc_charset.h> /* EnsureUTF8 */
> -#include <vlc_meta.h> /* vlc_meta_t,
> vlc_meta_ */
> -#include <vlc_input.h>
Removing so many headers cannot be right.
Removing <vlc_common.h> cannot be right.
--
Rémi Denis-Courmont
Sent from my collocated server
More information about the vlc-devel
mailing list