Move xiph metadata parsing to a .c|.h files couple

diff --git a/modules/demux/Modules.am b/modules/demux/Modules.am
index b52c9888c506f0b0af8cbc18b3e73205e73d8897..bcd01c6a5d0b3ccbcf07906b776d12d3023cd4dd 100644 (file)
--- a/modules/demux/Modules.am
+++ b/modules/demux/Modules.am
@@ -1,5 +1,5 @@
-SOURCES_flacsys = flac.c
-SOURCES_ogg = ogg.c ogg.h oggseek.c oggseek.h vorbis.h xiph.h
+SOURCES_flacsys = flac.c xiph_metadata.h xiph_metadata.c
+SOURCES_ogg = ogg.c ogg.h oggseek.c oggseek.h xiph_metadata.h xiph.h xiph_metadata.c
 SOURCES_demuxdump = demuxdump.c
 SOURCES_rawdv = rawdv.c rawdv.h
 SOURCES_rawvid = rawvid.c

diff --git a/modules/demux/flac.c b/modules/demux/flac.c
index 2c16272d9e84c3f396fe826cee2fc2e65b488dd9..2740f96c3dedfe08246aa880f4b27534fdc317c9 100644 (file)
--- a/modules/demux/flac.c
+++ b/modules/demux/flac.c
@@ -38,7 +38,7 @@
 #include <vlc_charset.h>              /* EnsureUTF8 */
 
 #include <assert.h>
-#include "vorbis.h"                   /* vorbis comments */
+#include "xiph_metadata.h"            /* vorbis comments */
 
 /*****************************************************************************
  * Module descriptor

diff --git a/modules/demux/ogg.c b/modules/demux/ogg.c
index 40f7161e382b0b8ca215584e6de57aa85d84bd83..b5750c3d1f536686101685945efadcc013901118 100644 (file)
--- a/modules/demux/ogg.c
+++ b/modules/demux/ogg.c
@@ -40,7 +40,7 @@
 #include <vlc_codecs.h>
 #include <vlc_bits.h>
 #include "xiph.h"
-#include "vorbis.h"
+#include "xiph_metadata.h"
 #include "ogg.h"
 #include "oggseek.h"
 

diff --git a/modules/demux/vorbis.h b/modules/demux/xiph_metadata.c
similarity index 89%
rename from modules/demux/vorbis.h
rename to modules/demux/xiph_metadata.c
index 379bd957a86f37087dc1370f6bb2484dc55ee598..75bf95b1b277fb2f8da3b8090f0c5829b3374f41 100644 (file)
--- a/modules/demux/vorbis.h
+++ b/modules/demux/xiph_metadata.c
@@ -1,5 +1,5 @@
 /*****************************************************************************
- * vorbis.h: Vorbis Comment parser
+ * xiph_metadata.h: Vorbis Comment parser
  *****************************************************************************
  * Copyright © 2008-2013 VLC authors and VideoLAN
  * $Id$
@@ -22,11 +22,17 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
  *****************************************************************************/
 
+#ifdef HAVE_CONFIG_H
+# include "config.h"
+#endif
+
+#include <vlc_common.h>
 #include <vlc_charset.h>
 #include <vlc_strings.h>
 #include <vlc_input.h>
+#include "xiph_metadata.h"
 
-static input_attachment_t* ParseFlacPicture( const uint8_t *p_data, int i_data,
+input_attachment_t* ParseFlacPicture( const uint8_t *p_data, int i_data,
     int i_attachments, int *i_cover_score, int *i_cover_idx )
 {
     /* TODO: Merge with ID3v2 copy in modules/meta_engine/taglib.cpp. */
@@ -106,7 +112,7 @@ error:
     return p_attachment;
 }
 
-static inline void vorbis_ParseComment( vlc_meta_t **pp_meta,
+void vorbis_ParseComment( vlc_meta_t **pp_meta,
         const uint8_t *p_data, int i_data,
         int *i_attachments, input_attachment_t ***attachments,
         int *i_cover_score, int *i_cover_idx,
@@ -266,35 +272,6 @@ static inline void vorbis_ParseComment( vlc_meta_t **pp_meta,
 #undef RM
 }
 
-static const struct {
-  const char *psz_tag;
-  const char *psz_i18n;
-} Katei18nCategories[] = {
-    /* From Silvia's Mozilla list */
-    { "CC",      N_("Closed captions") },
-    { "SUB",     N_("Subtitles") },
-    { "TAD",     N_("Textual audio descriptions") },
-    { "KTV",     N_("Karaoke") },
-    { "TIK",     N_("Ticker text") },
-    { "AR",      N_("Active regions") },
-    { "NB",      N_("Semantic annotations") },
-    { "META",    N_("Metadata") },
-    { "TRX",     N_("Transcript") },
-    { "LRC",     N_("Lyrics") },
-    { "LIN",     N_("Linguistic markup") },
-    { "CUE",     N_("Cue points") },
-
-    /* Grandfathered */
-    { "subtitles", N_("Subtitles") },
-    { "spu-subtitles", N_("Subtitles (images)") },
-    { "lyrics", N_("Lyrics") },
-
-    /* Kate specific */
-    { "K-SPU", N_("Subtitles (images)") },
-    { "K-SLD-T", N_("Slides (text)") },
-    { "K-SLD-I", N_("Slides (images)") },
-};
-
 const char *FindKateCategoryName( const char *psz_tag )
 {
     for( size_t i = 0; i < sizeof(Katei18nCategories)/sizeof(Katei18nCategories[0]); i++ )

diff --git a/modules/demux/xiph_metadata.h b/modules/demux/xiph_metadata.h
new file mode 100644 (file)
index 0000000..de97c2b
--- /dev/null
+++ b/modules/demux/xiph_metadata.h
@@ -0,0 +1,75 @@
+/*****************************************************************************
+ * xiph_metadata.h: Vorbis Comment parser
+ *****************************************************************************
+ * Copyright © 2008-2013 VLC authors and VideoLAN
+ * $Id$
+ *
+ * Authors: Laurent Aimar <fenrir _AT_ videolan _DOT_ org>
+ *          Jean-Baptiste Kempf <jb@videolan.org>
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU Lesser General Public License as published by
+ * the Free Software Foundation; either version 2.1 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with this program; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
+ *****************************************************************************/
+
+#include <vlc_charset.h>
+#include <vlc_strings.h>
+
+# ifdef __cplusplus
+extern "C" {
+# endif
+
+input_attachment_t* ParseFlacPicture( const uint8_t *p_data, int i_data,
+    int i_attachments, int *i_cover_score, int *i_cover_idx );
+
+void vorbis_ParseComment( vlc_meta_t **pp_meta,
+        const uint8_t *p_data, int i_data,
+        int *i_attachments, input_attachment_t ***attachments,
+        int *i_cover_score, int *i_cover_idx,
+        int *i_seekpoint, seekpoint_t ***ppp_seekpoint );
+
+static const struct {
+  const char *psz_tag;
+  const char *psz_i18n;
+} Katei18nCategories[] = {
+    /* From Silvia's Mozilla list */
+    { "CC",      N_("Closed captions") },
+    { "SUB",     N_("Subtitles") },
+    { "TAD",     N_("Textual audio descriptions") },
+    { "KTV",     N_("Karaoke") },
+    { "TIK",     N_("Ticker text") },
+    { "AR",      N_("Active regions") },
+    { "NB",      N_("Semantic annotations") },
+    { "META",    N_("Metadata") },
+    { "TRX",     N_("Transcript") },
+    { "LRC",     N_("Lyrics") },
+    { "LIN",     N_("Linguistic markup") },
+    { "CUE",     N_("Cue points") },
+
+    /* Grandfathered */
+    { "subtitles", N_("Subtitles") },
+    { "spu-subtitles", N_("Subtitles (images)") },
+    { "lyrics", N_("Lyrics") },
+
+    /* Kate specific */
+    { "K-SPU", N_("Subtitles (images)") },
+    { "K-SLD-T", N_("Slides (text)") },
+    { "K-SLD-I", N_("Slides (images)") },
+};
+
+const char *FindKateCategoryName( const char *psz_tag );
+
+# ifdef __cplusplus
+}
+# endif
+

diff --git a/modules/meta_engine/Modules.am b/modules/meta_engine/Modules.am
index f9fdcf17af6bce5a76b2c60de0427bc505bc5613..b9fd77514f8ab9a24c62b70be60d7f2ee46f90e8 100644 (file)
--- a/modules/meta_engine/Modules.am
+++ b/modules/meta_engine/Modules.am
@@ -1,5 +1,5 @@
 SOURCES_folder = folder.c
-SOURCES_taglib = taglib.cpp
+SOURCES_taglib = taglib.cpp ../demux/xiph_metadata.h ../demux/xiph_metadata.c
 
 libvlc_LTLIBRARIES += \
        libfolder_plugin.la

diff --git a/modules/meta_engine/taglib.cpp b/modules/meta_engine/taglib.cpp
index 76c878535c495c2af4192c47b79ead6caaae4cb0..8867514d2e7879c2704a540c160ca297714b9453 100644 (file)
--- a/modules/meta_engine/taglib.cpp
+++ b/modules/meta_engine/taglib.cpp
@@ -75,7 +75,7 @@
 #include <mpegfile.h>
 #include <oggfile.h>
 #include <oggflacfile.h>
-#include "../demux/vorbis.h"
+#include "../demux/xiph_metadata.h"
 
 #include <aifffile.h>
 #include <wavfile.h>