X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=modules%2Fdemux%2Fflac.c;h=b4beb1282be5118a4361a5276ee29693a84cdbc3;hb=c8d812a3393bd3c84feea0b50c608be2d4f95135;hp=89c94c276b5e5c6948c0bb7f6b946dd16720e5f0;hpb=85b29bdc288a1573d43bd524908be5748a9b3640;p=vlc diff --git a/modules/demux/flac.c b/modules/demux/flac.c index 89c94c276b..b4beb1282b 100644 --- a/modules/demux/flac.c +++ b/modules/demux/flac.c @@ -1,32 +1,44 @@ /***************************************************************************** * flac.c : FLAC demux module for vlc ***************************************************************************** - * Copyright (C) 2001-2003 VideoLAN (Centrale Réseaux) and its contributors + * Copyright (C) 2001-2008 VLC authors and VideoLAN * $Id$ * * Authors: Gildas Bazin + * Laurent Aimar * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation; either version 2 of the License, or + * 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 General Public License for more details. + * 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 General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111, USA. + * 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. *****************************************************************************/ /***************************************************************************** * Preamble *****************************************************************************/ -#include -#include -#include +#ifdef HAVE_CONFIG_H +# include "config.h" +#endif + +#include +#include +#include +#include /* vlc_meta_* */ +#include /* vlc_input_attachment, vlc_seekpoint */ +#include /* decoder_t */ +#include /* EnsureUTF8 */ + +#include +#include "xiph_metadata.h" /* vorbis comments */ /***************************************************************************** * Module descriptor @@ -34,14 +46,14 @@ static int Open ( vlc_object_t * ); static void Close ( vlc_object_t * ); -vlc_module_begin(); - set_description( _("FLAC demuxer") ); - set_capability( "demux2", 155 ); - set_category( CAT_INPUT ); - set_subcategory( SUBCAT_INPUT_DEMUX ); - set_callbacks( Open, Close ); - add_shortcut( "flac" ); -vlc_module_end(); +vlc_module_begin () + set_description( N_("FLAC demuxer") ) + set_capability( "demux", 155 ) + set_category( CAT_INPUT ) + set_subcategory( SUBCAT_INPUT_DEMUX ) + set_callbacks( Open, Close ) + add_shortcut( "flac" ) +vlc_module_end () /***************************************************************************** * Local prototypes @@ -49,14 +61,34 @@ vlc_module_end(); static int Demux ( demux_t * ); static int Control( demux_t *, int, va_list ); +static int ReadMeta( demux_t *, uint8_t **pp_streaminfo, int *pi_streaminfo ); + struct demux_sys_t { - vlc_bool_t b_start; + bool b_start; es_out_id_t *p_es; /* Packetizer */ decoder_t *p_packetizer; + vlc_meta_t *p_meta; + + int64_t i_time_offset; + int64_t i_pts; + int64_t i_pts_start; + + int64_t i_length; /* Length from stream info */ + int64_t i_data_pos; + + /* */ + int i_seekpoint; + seekpoint_t **seekpoint; + + /* */ + int i_attachments; + input_attachment_t **attachments; + int i_cover_idx; + int i_cover_score; }; #define STREAMINFO_SIZE 38 @@ -68,11 +100,11 @@ struct demux_sys_t static int Open( vlc_object_t * p_this ) { demux_t *p_demux = (demux_t*)p_this; - module_t *p_id3; demux_sys_t *p_sys; - int i_peek; - byte_t *p_peek; - es_format_t fmt; + const uint8_t *p_peek; + uint8_t *p_streaminfo; + int i_streaminfo; + es_format_t fmt; /* Have a peep at the show. */ if( stream_Peek( p_demux->s, &p_peek, 4 ) < 4 ) return VLC_EGENERIC; @@ -86,74 +118,55 @@ static int Open( vlc_object_t * p_this ) "continuing anyway" ); } + p_sys = malloc( sizeof( demux_sys_t ) ); + if( unlikely(p_sys == NULL) ) + return VLC_ENOMEM; + p_demux->pf_demux = Demux; p_demux->pf_control = Control; - p_demux->p_sys = p_sys = malloc( sizeof( demux_sys_t ) ); - es_format_Init( &fmt, AUDIO_ES, VLC_FOURCC( 'f', 'l', 'a', 'c' ) ); - p_sys->b_start = VLC_TRUE; - p_sys->p_meta = 0; + p_demux->p_sys = p_sys; + p_sys->b_start = true; + p_sys->p_meta = NULL; + p_sys->i_length = 0; + p_sys->i_time_offset = 0; + p_sys->i_pts = 0; + p_sys->i_pts_start = 0; + p_sys->p_es = NULL; + TAB_INIT( p_sys->i_seekpoint, p_sys->seekpoint ); + TAB_INIT( p_sys->i_attachments, p_sys->attachments); + p_sys->i_cover_idx = 0; + p_sys->i_cover_score = 0; /* We need to read and store the STREAMINFO metadata */ - i_peek = stream_Peek( p_demux->s, &p_peek, 8 ); - if( p_peek[4] & 0x7F ) + if( ReadMeta( p_demux, &p_streaminfo, &i_streaminfo ) ) { - msg_Err( p_demux, "this isn't a STREAMINFO metadata block" ); + free( p_sys ); return VLC_EGENERIC; } - if( ((p_peek[5]<<16)+(p_peek[6]<<8)+p_peek[7]) != (STREAMINFO_SIZE - 4) ) - { - msg_Err( p_demux, "invalid size for a STREAMINFO metadata block" ); - return VLC_EGENERIC; - } - - /* - * Load the FLAC packetizer - */ - p_sys->p_packetizer = vlc_object_create( p_demux, VLC_OBJECT_DECODER ); - p_sys->p_packetizer->pf_decode_audio = 0; - p_sys->p_packetizer->pf_decode_video = 0; - p_sys->p_packetizer->pf_decode_sub = 0; - p_sys->p_packetizer->pf_packetize = 0; - - /* Initialization of decoder structure */ - es_format_Init( &p_sys->p_packetizer->fmt_in, AUDIO_ES, - VLC_FOURCC( 'f', 'l', 'a', 'c' ) ); - + /* Load the FLAC packetizer */ /* Store STREAMINFO for the decoder and packetizer */ - p_sys->p_packetizer->fmt_in.i_extra = fmt.i_extra = STREAMINFO_SIZE + 4; - p_sys->p_packetizer->fmt_in.p_extra = malloc( STREAMINFO_SIZE + 4 ); - stream_Read( p_demux->s, p_sys->p_packetizer->fmt_in.p_extra, - STREAMINFO_SIZE + 4 ); - - /* Fake this as the last metadata block */ - ((uint8_t*)p_sys->p_packetizer->fmt_in.p_extra)[4] |= 0x80; - fmt.p_extra = malloc( STREAMINFO_SIZE + 4 ); - memcpy( fmt.p_extra, p_sys->p_packetizer->fmt_in.p_extra, - STREAMINFO_SIZE + 4 ); - - p_sys->p_packetizer->p_module = - module_Need( p_sys->p_packetizer, "packetizer", NULL, 0 ); - if( !p_sys->p_packetizer->p_module ) - { - if( p_sys->p_packetizer->fmt_in.p_extra ) - free( p_sys->p_packetizer->fmt_in.p_extra ); - vlc_object_destroy( p_sys->p_packetizer ); - - msg_Err( p_demux, "cannot find flac packetizer" ); + p_streaminfo[4] |= 0x80; /* Fake this as the last metadata block */ + es_format_Init( &fmt, AUDIO_ES, VLC_CODEC_FLAC ); + fmt.i_extra = i_streaminfo; + fmt.p_extra = p_streaminfo; + + p_sys->p_packetizer = demux_PacketizerNew( p_demux, &fmt, "flac" ); + if( !p_sys->p_packetizer ) + { + free( p_sys ); return VLC_EGENERIC; } - p_sys->p_es = es_out_Add( p_demux->out, &fmt ); - - /* Parse possible id3 header */ - if( ( p_id3 = module_Need( p_demux, "id3", NULL, 0 ) ) ) + if( p_sys->i_cover_idx < p_sys->i_attachments ) { - p_sys->p_meta = (vlc_meta_t *)p_demux->p_private; - p_demux->p_private = NULL; - module_Unneed( p_demux, p_id3 ); + char psz_url[128]; + if( !p_sys->p_meta ) + p_sys->p_meta = vlc_meta_New(); + snprintf( psz_url, sizeof(psz_url), "attachment://%s", + p_sys->attachments[p_sys->i_cover_idx]->psz_name ); + vlc_meta_Set( p_sys->p_meta, vlc_meta_ArtworkURL, psz_url ); } - return VLC_SUCCESS; } @@ -165,15 +178,18 @@ static void Close( vlc_object_t * p_this ) demux_t *p_demux = (demux_t*)p_this; demux_sys_t *p_sys = p_demux->p_sys; - /* Unneed module */ - module_Unneed( p_sys->p_packetizer, p_sys->p_packetizer->p_module ); + TAB_CLEAN( p_sys->i_seekpoint, p_sys->seekpoint ); - if( p_sys->p_packetizer->fmt_in.p_extra ) - free( p_sys->p_packetizer->fmt_in.p_extra ); + int i; + for( i = 0; i < p_sys->i_attachments; i++ ) + vlc_input_attachment_Delete( p_sys->attachments[i] ); + TAB_CLEAN( p_sys->i_attachments, p_sys->attachments); /* Delete the decoder */ - vlc_object_destroy( p_sys->p_packetizer ); - if( p_sys->p_meta ) vlc_meta_Delete( p_sys->p_meta ); + demux_PacketizerDestroy( p_sys->p_packetizer ); + + if( p_sys->p_meta ) + vlc_meta_Delete( p_sys->p_meta ); free( p_sys ); } @@ -188,19 +204,10 @@ static int Demux( demux_t *p_demux ) block_t *p_block_in, *p_block_out; if( !( p_block_in = stream_Block( p_demux->s, FLAC_PACKET_SIZE ) ) ) - { return 0; - } - if( p_sys->b_start ) - { - p_block_in->i_pts = p_block_in->i_dts = 1; - p_sys->b_start = VLC_FALSE; - } - else - { - p_block_in->i_pts = p_block_in->i_dts = 0; - } + p_block_in->i_pts = p_block_in->i_dts = p_sys->b_start ? VLC_TS_0 : VLC_TS_INVALID; + p_sys->b_start = false; while( (p_block_out = p_sys->p_packetizer->pf_packetize( p_sys->p_packetizer, &p_block_in )) ) @@ -209,6 +216,20 @@ static int Demux( demux_t *p_demux ) { block_t *p_next = p_block_out->p_next; + p_block_out->p_next = NULL; + + if( p_sys->p_es == NULL ) + { + p_sys->p_packetizer->fmt_out.b_packetized = true; + p_sys->p_es = es_out_Add( p_demux->out, &p_sys->p_packetizer->fmt_out); + } + + p_sys->i_pts = p_block_out->i_dts - VLC_TS_0; + + /* Correct timestamp */ + p_block_out->i_pts += p_sys->i_time_offset; + p_block_out->i_dts += p_sys->i_time_offset; + /* set PCR */ es_out_Control( p_demux->out, ES_OUT_SET_PCR, p_block_out->i_dts ); @@ -217,27 +238,368 @@ static int Demux( demux_t *p_demux ) p_block_out = p_next; } } - return 1; } /***************************************************************************** * Control: *****************************************************************************/ +static int64_t ControlGetLength( demux_t *p_demux ) +{ + demux_sys_t *p_sys = p_demux->p_sys; + const int64_t i_size = stream_Size(p_demux->s) - p_sys->i_data_pos; + int64_t i_length = p_sys->i_length; + int i; + + /* Try to fix length using seekpoint and current size for truncated file */ + for( i = p_sys->i_seekpoint-1; i >= 0; i-- ) + { + seekpoint_t *s = p_sys->seekpoint[i]; + if( s->i_byte_offset <= i_size ) + { + if( i+1 < p_sys->i_seekpoint ) + { + /* Broken file */ + seekpoint_t *n = p_sys->seekpoint[i+1]; + assert( n->i_byte_offset != s->i_byte_offset); /* Should be ensured by ParseSeekTable */ + i_length = s->i_time_offset + (n->i_time_offset-s->i_time_offset) * (i_size-s->i_byte_offset) / (n->i_byte_offset-s->i_byte_offset); + } + break; + } + } + return i_length; +} + +static int64_t ControlGetTime( demux_t *p_demux ) +{ + demux_sys_t *p_sys = p_demux->p_sys; + return __MAX(p_sys->i_pts, p_sys->i_pts_start) + p_sys->i_time_offset; +} + +static int ControlSetTime( demux_t *p_demux, int64_t i_time ) +{ + demux_sys_t *p_sys = p_demux->p_sys; + int64_t i_delta_time; + bool b_seekable; + int i; + + /* */ + stream_Control( p_demux->s, STREAM_CAN_SEEK, &b_seekable ); + if( !b_seekable ) + return VLC_EGENERIC; + + /* */ + assert( p_sys->i_seekpoint > 0 ); /* ReadMeta ensure at least (0,0) */ + for( i = p_sys->i_seekpoint-1; i >= 0; i-- ) + { + if( p_sys->seekpoint[i]->i_time_offset <= i_time ) + break; + } + i_delta_time = i_time - p_sys->seekpoint[i]->i_time_offset; + + /* XXX We do exact seek if it's not too far away(45s) */ + if( i_delta_time < 45*INT64_C(1000000) ) + { + if( stream_Seek( p_demux->s, p_sys->seekpoint[i]->i_byte_offset+p_sys->i_data_pos ) ) + return VLC_EGENERIC; + + p_sys->i_time_offset = p_sys->seekpoint[i]->i_time_offset - p_sys->i_pts; + p_sys->i_pts_start = p_sys->i_pts+i_delta_time; + es_out_Control( p_demux->out, ES_OUT_SET_NEXT_DISPLAY_TIME, p_sys->i_pts_start + p_sys->i_time_offset ); + } + else + { + int64_t i_delta_offset; + int64_t i_next_time; + int64_t i_next_offset; + + if( i+1 < p_sys->i_seekpoint ) + { + i_next_time = p_sys->seekpoint[i+1]->i_time_offset; + i_next_offset = p_sys->seekpoint[i+1]->i_byte_offset; + } + else + { + i_next_time = p_sys->i_length; + i_next_offset = stream_Size(p_demux->s)-p_sys->i_data_pos; + } + + i_delta_offset = 0; + if( i_next_time-p_sys->seekpoint[i]->i_time_offset > 0 ) + i_delta_offset = (i_next_offset - p_sys->seekpoint[i]->i_byte_offset) * i_delta_time / + (i_next_time-p_sys->seekpoint[i]->i_time_offset); + + if( stream_Seek( p_demux->s, p_sys->seekpoint[i]->i_byte_offset+p_sys->i_data_pos + i_delta_offset ) ) + return VLC_EGENERIC; + + p_sys->i_pts_start = p_sys->i_pts; + p_sys->i_time_offset = (p_sys->seekpoint[i]->i_time_offset+i_delta_time) - p_sys->i_pts; + } + return VLC_SUCCESS; +} + static int Control( demux_t *p_demux, int i_query, va_list args ) { - /* demux_sys_t *p_sys = p_demux->p_sys; */ - /* FIXME bitrate */ - if( i_query == DEMUX_SET_TIME ) return VLC_EGENERIC; - else if( i_query == DEMUX_GET_META ) - { - vlc_meta_t **pp_meta = (vlc_meta_t **)va_arg( args, vlc_meta_t** ); + demux_sys_t *p_sys = p_demux->p_sys; + + if( i_query == DEMUX_GET_META ) + { + vlc_meta_t *p_meta = (vlc_meta_t *)va_arg( args, vlc_meta_t* ); if( p_demux->p_sys->p_meta ) - *pp_meta = vlc_meta_Duplicate( p_demux->p_sys->p_meta ); - else *pp_meta = NULL; + vlc_meta_Merge( p_meta, p_demux->p_sys->p_meta ); + return VLC_SUCCESS; + } + else if( i_query == DEMUX_HAS_UNSUPPORTED_META ) + { + bool *pb_bool = (bool*)va_arg( args, bool* ); + *pb_bool = true; return VLC_SUCCESS; } - else return demux2_vaControlHelper( p_demux->s, 0, -1, - 8*0, 1, i_query, args ); + else if( i_query == DEMUX_GET_LENGTH ) + { + int64_t *pi64 = (int64_t*)va_arg( args, int64_t * ); + *pi64 = ControlGetLength( p_demux ); + return VLC_SUCCESS; + } + else if( i_query == DEMUX_SET_TIME ) + { + int64_t i_time = (int64_t)va_arg( args, int64_t ); + return ControlSetTime( p_demux, i_time ); + } + else if( i_query == DEMUX_SET_POSITION ) + { + const double f = (double)va_arg( args, double ); + int64_t i_time = f * ControlGetLength( p_demux ); + return ControlSetTime( p_demux, i_time ); + } + else if( i_query == DEMUX_GET_TIME ) + { + int64_t *pi64 = (int64_t*)va_arg( args, int64_t * ); + *pi64 = ControlGetTime( p_demux ); + return VLC_SUCCESS; + } + else if( i_query == DEMUX_GET_POSITION ) + { + double *pf = (double*)va_arg( args, double * ); + const int64_t i_length = ControlGetLength(p_demux); + if( i_length > 0 ) + { + double current = ControlGetTime(p_demux); + *pf = current / (double)i_length; + } + else + *pf= 0.0; + return VLC_SUCCESS; + } + else if( i_query == DEMUX_GET_ATTACHMENTS ) + { + input_attachment_t ***ppp_attach = + (input_attachment_t***)va_arg( args, input_attachment_t*** ); + int *pi_int = (int*)va_arg( args, int * ); + + if( p_sys->i_attachments <= 0 ) + return VLC_EGENERIC; + + *pi_int = p_sys->i_attachments; + *ppp_attach = xmalloc( sizeof(input_attachment_t**) * p_sys->i_attachments ); + for( int i = 0; i < p_sys->i_attachments; i++ ) + (*ppp_attach)[i] = vlc_input_attachment_Duplicate( p_sys->attachments[i] ); + return VLC_SUCCESS; + } + + return demux_vaControlHelper( p_demux->s, p_sys->i_data_pos, -1, + 8*0, 1, i_query, args ); +} + +enum +{ + META_STREAMINFO = 0, + META_SEEKTABLE = 3, + META_COMMENT = 4, + META_PICTURE = 6, +}; + +static inline int Get24bBE( const uint8_t *p ) +{ + return (p[0] << 16)|(p[1] << 8)|(p[2]); +} + +static void ParseStreamInfo( int *pi_rate, int64_t *pi_count, uint8_t *p_data ); +static void ParseSeekTable( demux_t *p_demux, const uint8_t *p_data, int i_data, + int i_sample_rate ); +static void ParseComment( demux_t *, const uint8_t *p_data, int i_data ); +static void ParsePicture( demux_t *, const uint8_t *p_data, int i_data ); + +static int ReadMeta( demux_t *p_demux, uint8_t **pp_streaminfo, int *pi_streaminfo ) +{ + demux_sys_t *p_sys = p_demux->p_sys; + int i_peek; + const uint8_t *p_peek; + bool b_last; + int i_sample_rate = 0; + int64_t i_sample_count; + + *pp_streaminfo = NULL; + + /* Be sure we have seekpoint 0 */ + seekpoint_t *s = vlc_seekpoint_New(); + s->i_time_offset = 0; + s->i_byte_offset = 0; + TAB_APPEND( p_sys->i_seekpoint, p_sys->seekpoint, s ); + + static const uint8_t marker[4] = { 'f', 'L', 'a', 'C' }; + uint8_t header[4]; + if( stream_Read( p_demux->s, header, 4) < 4) + return VLC_EGENERIC; + + if (memcmp(header, marker, 4)) + return VLC_EGENERIC; + + b_last = 0; + while( !b_last ) + { + int i_len; + int i_type; + + i_peek = stream_Peek( p_demux->s, &p_peek, 4 ); + if( i_peek < 4 ) + break; + b_last = p_peek[0]&0x80; + i_type = p_peek[0]&0x7f; + i_len = Get24bBE( &p_peek[1] ); + + if( i_type == META_STREAMINFO && !*pp_streaminfo ) + { + if( i_len != (STREAMINFO_SIZE - 4) ) { + msg_Err( p_demux, "invalid size %d for a STREAMINFO metadata block", i_len ); + return VLC_EGENERIC; + } + i_peek = stream_Peek( p_demux->s, &p_peek, STREAMINFO_SIZE); + if( i_peek == STREAMINFO_SIZE) + + *pi_streaminfo = STREAMINFO_SIZE + 4; + *pp_streaminfo = malloc( STREAMINFO_SIZE + 4 ); + if( *pp_streaminfo == NULL ) + return VLC_EGENERIC; + + if( stream_Read( p_demux->s, &(*pp_streaminfo)[4], STREAMINFO_SIZE ) != STREAMINFO_SIZE ) + { + msg_Err( p_demux, "failed to read STREAMINFO metadata block" ); + free( *pp_streaminfo ); + return VLC_EGENERIC; + } + + memcpy(*pp_streaminfo, marker, 4); + + /* */ + ParseStreamInfo( &i_sample_rate, &i_sample_count, *pp_streaminfo ); + if( i_sample_rate > 0 ) + p_sys->i_length = i_sample_count * INT64_C(1000000)/i_sample_rate; + } + else if( i_type == META_SEEKTABLE ) + { + i_peek = stream_Peek( p_demux->s, &p_peek, 4+i_len ); + if( i_peek == 4+i_len ) + ParseSeekTable( p_demux, p_peek, i_peek, i_sample_rate ); + } + else if( i_type == META_COMMENT ) + { + i_peek = stream_Peek( p_demux->s, &p_peek, 4+i_len ); + if( i_peek == 4+i_len ) + ParseComment( p_demux, p_peek, i_peek ); + } + else if( i_type == META_PICTURE ) + { + i_peek = stream_Peek( p_demux->s, &p_peek, 4+i_len ); + if( i_peek == 4+i_len ) + ParsePicture( p_demux, p_peek, i_peek ); + } + + if( stream_Read( p_demux->s, NULL, 4+i_len ) < 4+i_len ) + break; + } + + /* */ + p_sys->i_data_pos = stream_Tell( p_demux->s ); + + if (!*pp_streaminfo) + return VLC_EGENERIC; + + return VLC_SUCCESS; +} +static void ParseStreamInfo( int *pi_rate, int64_t *pi_count, uint8_t *p_data ) +{ + const int i_skip = 4+4; + + *pi_rate = GetDWBE(&p_data[i_skip+4+6]) >> 12; + *pi_count = GetQWBE(&p_data[i_skip+4+6]) & ((INT64_C(1)<<36)-1); +} + +static void ParseSeekTable( demux_t *p_demux, const uint8_t *p_data, int i_data, + int i_sample_rate ) +{ + demux_sys_t *p_sys = p_demux->p_sys; + seekpoint_t *s; + int i; + + if( i_sample_rate <= 0 ) + return; + + /* */ + for( i = 0; i < (i_data-4)/18; i++ ) + { + const int64_t i_sample = GetQWBE( &p_data[4+18*i+0] ); + int j; + + if( i_sample < 0 || i_sample >= INT64_MAX ) + continue; + + s = vlc_seekpoint_New(); + s->i_time_offset = i_sample * INT64_C(1000000)/i_sample_rate; + s->i_byte_offset = GetQWBE( &p_data[4+18*i+8] ); + + /* Check for duplicate entry */ + for( j = 0; j < p_sys->i_seekpoint; j++ ) + { + if( p_sys->seekpoint[j]->i_time_offset == s->i_time_offset || + p_sys->seekpoint[j]->i_byte_offset == s->i_byte_offset ) + { + vlc_seekpoint_Delete( s ); + s = NULL; + break; + } + } + if( s ) + { + TAB_APPEND( p_sys->i_seekpoint, p_sys->seekpoint, s ); + } + } + /* TODO sort it by size and remove wrong seek entry (time not increasing) */ +} + +static void ParseComment( demux_t *p_demux, const uint8_t *p_data, int i_data ) +{ + demux_sys_t *p_sys = p_demux->p_sys; + + if( i_data < 4 ) + return; + + vorbis_ParseComment( &p_sys->p_meta, &p_data[4], i_data - 4, + &p_sys->i_attachments, &p_sys->attachments, + &p_sys->i_cover_score, &p_sys->i_cover_idx, NULL, NULL ); +} + +static void ParsePicture( demux_t *p_demux, const uint8_t *p_data, int i_data ) +{ + demux_sys_t *p_sys = p_demux->p_sys; + + i_data -= 4; p_data += 4; + + input_attachment_t *p_attachment = ParseFlacPicture( p_data, i_data, + p_sys->i_attachments, &p_sys->i_cover_score, &p_sys->i_cover_idx ); + if( p_attachment == NULL ) + return; + + TAB_APPEND( p_sys->i_attachments, p_sys->attachments, p_attachment ); }