X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=src%2Finput%2Fstream.c;h=d21eeb7af4fa80489cfe9aba7fe484a000019afd;hb=39670df97892b3ef4be6edd28e39826804f51cff;hp=b3447af94af4afe7ff892d0e284f727b31892359;hpb=449fd28aaf007c6411251dae9d0dbfdc65b135d1;p=vlc diff --git a/src/input/stream.c b/src/input/stream.c index b3447af94a..d21eeb7af4 100644 --- a/src/input/stream.c +++ b/src/input/stream.c @@ -25,17 +25,20 @@ # include "config.h" #endif -#include +#include + +#include #include "input_internal.h" #undef STREAM_DEBUG /* TODO: - * - tune the 2 methods + * - tune the 2 methods (block/stream) * - compute cost for seek * - improve stream mode seeking with closest segments * - ... + * - Maybe remove (block/stream) in favour of immediate */ /* Two methods: @@ -43,6 +46,9 @@ * One linked list of data read * - using pf_read * More complex scheme using mutliple track to avoid seeking + * - using directly the access (only indirection for peeking). + * This method is known to introduce much less latency. + * It should probably defaulted (instead of the stream method (2)). */ /* How many tracks we have, currently only used for stream mode */ @@ -101,11 +107,18 @@ typedef struct } access_entry_t; +typedef enum stream_read_method_t +{ + Immediate, + Block, + Stream +} stream_read_method_t; + struct stream_sys_t { access_t *p_access; - bool b_block; /* Block method (1) or stream */ + stream_read_method_t method; /* method to use */ int64_t i_pos; /* Current reading offset */ @@ -138,6 +151,13 @@ struct stream_sys_t } stream; + /* Method 3: for pf_read */ + struct + { + int64_t i_end; + uint8_t *p_buffer; + } immediate; + /* Peek temporary buffer */ int i_peek; uint8_t *p_peek; @@ -182,12 +202,47 @@ static int AStreamSeekStream( stream_t *s, int64_t i_pos ); static void AStreamPrebufferStream( stream_t *s ); static int AReadStream( stream_t *s, void *p_read, int i_read ); +/* Method 3 */ +static int AStreamReadImmediate( stream_t *s, void *p_read, int i_read ); +static int AStreamPeekImmediate( stream_t *s, const uint8_t **pp_peek, int i_read ); +static int AStreamSeekImmediate( stream_t *s, int64_t i_pos ); + /* Common */ static int AStreamControl( stream_t *s, int i_query, va_list ); static void AStreamDestroy( stream_t *s ); static void UStreamDestroy( stream_t *s ); static int ASeek( stream_t *s, int64_t i_pos ); +/**************************************************************************** + * Method 3 helpers: + ****************************************************************************/ + +static inline int64_t stream_buffered_size( stream_t *s ) +{ + return s->p_sys->immediate.i_end; +} + +static inline void stream_buffer_empty( stream_t *s, int length ) +{ + length = __MAX( stream_buffered_size( s ), length ); + if( length ) + { + memmove( s->p_sys->immediate.p_buffer, + s->p_sys->immediate.p_buffer + length, + stream_buffered_size( s ) - length ); + } + s->p_sys->immediate.i_end -= length; +} + +static inline void stream_buffer_fill( stream_t *s, int length ) +{ + s->p_sys->immediate.i_end += length; +} + +static inline uint8_t * stream_buffer( stream_t *s ) +{ + return s->p_sys->immediate.p_buffer; +} /**************************************************************************** * stream_UrlNew: create a stream from a access @@ -206,7 +261,7 @@ stream_t *__stream_UrlNew( vlc_object_t *p_parent, const char *psz_url ) MRLSplit( psz_dup, &psz_access, &psz_demux, &psz_path ); /* Now try a real access */ - p_access = access2_New( p_parent, psz_access, psz_demux, psz_path ); + p_access = access_New( p_parent, psz_access, psz_demux, psz_path ); if( p_access == NULL ) { @@ -216,7 +271,7 @@ stream_t *__stream_UrlNew( vlc_object_t *p_parent, const char *psz_url ) if( !( p_res = stream_AccessNew( p_access, true ) ) ) { - access2_Delete( p_access ); + access_Delete( p_access ); return NULL; } @@ -242,10 +297,7 @@ stream_t *stream_AccessNew( access_t *p_access, bool b_quick ) s->p_sys = p_sys = malloc( sizeof( stream_sys_t ) ); if( p_sys == NULL ) - { - msg_Err( s, "Out of memory when allocating stream_sys_t" ); goto error; - } /* UTF16 and UTF32 text file conversion */ s->i_char_width = 1; @@ -254,11 +306,17 @@ stream_t *stream_AccessNew( access_t *p_access, bool b_quick ) /* Common field */ p_sys->p_access = p_access; - p_sys->b_block = p_access->pf_block ? true : false; + if( p_access->pf_block ) + p_sys->method = Block; + else if (var_CreateGetBool( s, "use-stream-immediate")) + p_sys->method = Immediate; + else + p_sys->method = Stream; + p_sys->i_pos = p_access->info.i_pos; /* Stats */ - access2_Control( p_access, ACCESS_CAN_FASTSEEK, &p_sys->stat.b_fastseek ); + access_Control( p_access, ACCESS_CAN_FASTSEEK, &p_sys->stat.b_fastseek ); p_sys->stat.i_bytes = 0; p_sys->stat.i_read_time = 0; p_sys->stat.i_read_count = 0; @@ -277,10 +335,7 @@ stream_t *stream_AccessNew( access_t *p_access, bool b_quick ) { access_entry_t *p_entry = malloc( sizeof(access_entry_t) ); if( p_entry == NULL ) - { - msg_Err( s, "Out of memory when allocating access_entry_t" ); goto error; - } char *psz_name, *psz_parser = psz_name = psz_list; p_sys->p_list_access = p_access; @@ -288,12 +343,11 @@ stream_t *stream_AccessNew( access_t *p_access, bool b_quick ) p_entry->psz_path = strdup( p_access->psz_path ); if( p_entry->psz_path == NULL ) { - msg_Err( s, "Out of memory when duplicating p_access->psz_path" ); free( p_entry ); goto error; } TAB_APPEND( p_sys->i_list, p_sys->list, p_entry ); - msg_Dbg( p_access, "adding file `%s', ("I64Fd" bytes)", + msg_Dbg( p_access, "adding file `%s', (%"PRId64" bytes)", p_entry->psz_path, p_access->info.i_size ); while( psz_name && *psz_name ) @@ -304,7 +358,7 @@ stream_t *stream_AccessNew( access_t *p_access, bool b_quick ) psz_name = strdup( psz_name ); if( psz_name ) { - access_t *p_tmp = access2_New( p_access, p_access->psz_access, + access_t *p_tmp = access_New( p_access, p_access->psz_access, "", psz_name ); if( !p_tmp ) @@ -314,20 +368,17 @@ stream_t *stream_AccessNew( access_t *p_access, bool b_quick ) continue; } - msg_Dbg( p_access, "adding file `%s', ("I64Fd" bytes)", + msg_Dbg( p_access, "adding file `%s', (%"PRId64" bytes)", psz_name, p_tmp->info.i_size ); p_entry = malloc( sizeof(access_entry_t) ); if( p_entry == NULL ) - { - msg_Err( p_access, "Out of memory when allocating access_entry_t" ); goto error; - } p_entry->i_size = p_tmp->info.i_size; p_entry->psz_path = psz_name; TAB_APPEND( p_sys->i_list, p_sys->list, p_entry ); - access2_Delete( p_tmp ); + access_Delete( p_tmp ); } psz_name = psz_parser; @@ -340,8 +391,9 @@ stream_t *stream_AccessNew( access_t *p_access, bool b_quick ) p_sys->i_peek = 0; p_sys->p_peek = NULL; - if( p_sys->b_block ) + if( p_sys->method == Block ) { + msg_Dbg( s, "Using AStream*Block" ); s->pf_read = AStreamReadBlock; s->pf_peek = AStreamPeekBlock; @@ -362,10 +414,33 @@ stream_t *stream_AccessNew( access_t *p_access, bool b_quick ) goto error; } } - else + else if (p_sys->method == Immediate) + { + msg_Dbg( s, "Using AStream*Immediate" ); + + s->pf_read = AStreamReadImmediate; + s->pf_peek = AStreamPeekImmediate; + + /* Allocate/Setup our tracks (useful to peek)*/ + p_sys->immediate.i_end = 0; + p_sys->immediate.p_buffer = malloc( STREAM_CACHE_SIZE ); + + msg_Dbg( s, "p_buffer %p-%p", p_sys->immediate.p_buffer, + p_sys->immediate.p_buffer + STREAM_CACHE_SIZE ); + + if( p_sys->immediate.p_buffer == NULL ) + { + msg_Err( s, "Out of memory when allocating stream cache (%d bytes)", + STREAM_CACHE_SIZE ); + goto error; + } + } + else /* ( p_sys->method == Stream ) */ { int i; + msg_Dbg( s, "Using AStream*Stream" ); + s->pf_read = AStreamReadStream; s->pf_peek = AStreamPeekStream; @@ -380,7 +455,7 @@ stream_t *stream_AccessNew( access_t *p_access, bool b_quick ) goto error; } p_sys->stream.i_used = 0; - access2_Control( p_access, ACCESS_GET_MTU, + access_Control( p_access, ACCESS_GET_MTU, &p_sys->stream.i_read_size ); if( p_sys->stream.i_read_size <= 0 ) p_sys->stream.i_read_size = STREAM_READ_ATONCE; @@ -409,7 +484,7 @@ stream_t *stream_AccessNew( access_t *p_access, bool b_quick ) return s; error: - if( p_sys->b_block ) + if( p_sys->method == Block ) { /* Nothing yet */ } @@ -436,31 +511,31 @@ static void AStreamDestroy( stream_t *s ) vlc_object_detach( s ); - if( p_sys->b_block ) block_ChainRelease( p_sys->block.p_first ); + if( p_sys->method == Block ) block_ChainRelease( p_sys->block.p_first ); else free( p_sys->stream.p_buffer ); free( p_sys->p_peek ); if( p_sys->p_list_access && p_sys->p_list_access != p_sys->p_access ) - access2_Delete( p_sys->p_list_access ); + access_Delete( p_sys->p_list_access ); while( p_sys->i_list-- ) { free( p_sys->list[p_sys->i_list]->psz_path ); free( p_sys->list[p_sys->i_list] ); - free( p_sys->list ); } - free( s->p_sys ); + free( p_sys->list ); + free( p_sys ); + vlc_object_release( s ); } static void UStreamDestroy( stream_t *s ) { - access_t *p_access = (access_t*)vlc_object_find( s, VLC_OBJECT_ACCESS, FIND_PARENT ); + access_t *p_access = (access_t *)s->p_parent; AStreamDestroy( s ); - vlc_object_release( p_access ); - access2_Delete( p_access ); + access_Delete( p_access ); } /**************************************************************************** @@ -472,7 +547,7 @@ void stream_AccessReset( stream_t *s ) p_sys->i_pos = p_sys->p_access->info.i_pos; - if( p_sys->b_block ) + if( p_sys->method == Block ) { block_ChainRelease( p_sys->block.p_first ); @@ -487,7 +562,11 @@ void stream_AccessReset( stream_t *s ) /* Do the prebuffering */ AStreamPrebufferBlock( s ); } - else + else if( p_sys->method == Immediate ) + { + stream_buffer_empty( s, stream_buffered_size( s ) ); + } + else /* ( p_sys->method == Stream ) */ { int i; @@ -556,12 +635,12 @@ static int AStreamControl( stream_t *s, int i_query, va_list args ) case STREAM_CAN_SEEK: p_bool = (bool*)va_arg( args, bool * ); - access2_Control( p_access, ACCESS_CAN_SEEK, p_bool ); + access_Control( p_access, ACCESS_CAN_SEEK, p_bool ); break; case STREAM_CAN_FASTSEEK: p_bool = (bool*)va_arg( args, bool * ); - access2_Control( p_access, ACCESS_CAN_FASTSEEK, p_bool ); + access_Control( p_access, ACCESS_CAN_FASTSEEK, p_bool ); break; case STREAM_GET_POSITION: @@ -571,9 +650,11 @@ static int AStreamControl( stream_t *s, int i_query, va_list args ) case STREAM_SET_POSITION: i_64 = (int64_t)va_arg( args, int64_t ); - if( p_sys->b_block ) + if( p_sys->method == Block ) return AStreamSeekBlock( s, i_64 ); - else + else if( p_sys->method == Immediate ) + return AStreamSeekImmediate( s, i_64 ); + else /* ( p_sys->method == Stream ) */ return AStreamSeekStream( s, i_64 ); case STREAM_GET_MTU: @@ -589,10 +670,10 @@ static int AStreamControl( stream_t *s, int i_query, va_list args ) "DON'T USE STREAM_CONTROL_ACCESS !!!" ); return VLC_EGENERIC; } - return access2_vaControl( p_access, i_int, args ); + return access_vaControl( p_access, i_int, args ); case STREAM_GET_CONTENT_TYPE: - return access2_Control( p_access, ACCESS_GET_CONTENT_TYPE, + return access_Control( p_access, ACCESS_GET_CONTENT_TYPE, va_arg( args, char ** ) ); default: @@ -631,13 +712,13 @@ static void AStreamPrebufferBlock( stream_t *s ) /* Update stat */ p_sys->stat.i_bytes = p_sys->block.i_size; p_sys->stat.i_read_time = i_date - i_start; - i_byterate = ( I64C(1000000) * p_sys->stat.i_bytes ) / + i_byterate = ( INT64_C(1000000) * p_sys->stat.i_bytes ) / (p_sys->stat.i_read_time + 1); - msg_Dbg( s, "prebuffering done "I64Fd" bytes in "I64Fd"s - " - I64Fd" kbytes/s", + msg_Dbg( s, "prebuffering done %"PRId64" bytes in %"PRId64"s - " + "%"PRId64" kbytes/s", p_sys->stat.i_bytes, - p_sys->stat.i_read_time / I64C(1000000), + p_sys->stat.i_read_time / INT64_C(1000000), i_byterate / 1024 ); break; } @@ -700,7 +781,7 @@ static int AStreamReadBlock( stream_t *s, void *p_read, int i_read ) stream_sys_t *p_sys = s->p_sys; access_t *p_access = p_sys->p_access; bool b_aseek; - access2_Control( p_access, ACCESS_CAN_SEEK, &b_aseek ); + access_Control( p_access, ACCESS_CAN_SEEK, &b_aseek ); if( b_aseek ) return AStreamSeekBlock( s, p_sys->i_pos + i_read ) ? 0 : i_read; } @@ -840,7 +921,7 @@ static int AStreamSeekBlock( stream_t *s, int64_t i_pos ) if( i_offset < 0 ) { bool b_aseek; - access2_Control( p_access, ACCESS_CAN_SEEK, &b_aseek ); + access_Control( p_access, ACCESS_CAN_SEEK, &b_aseek ); if( !b_aseek ) { @@ -854,13 +935,13 @@ static int AStreamSeekBlock( stream_t *s, int64_t i_pos ) { bool b_aseek, b_aseekfast; - access2_Control( p_access, ACCESS_CAN_SEEK, &b_aseek ); - access2_Control( p_access, ACCESS_CAN_FASTSEEK, &b_aseekfast ); + access_Control( p_access, ACCESS_CAN_SEEK, &b_aseek ); + access_Control( p_access, ACCESS_CAN_FASTSEEK, &b_aseekfast ); if( !b_aseek ) { b_seek = false; - msg_Warn( s, I64Fd" bytes need to be skipped " + msg_Warn( s, "%"PRId64" bytes need to be skipped " "(access non seekable)", i_offset - p_sys->block.i_size ); } @@ -879,7 +960,7 @@ static int AStreamSeekBlock( stream_t *s, int64_t i_pos ) else b_seek = true; - msg_Dbg( s, "b_seek=%d th*avg=%d skip="I64Fd, + msg_Dbg( s, "b_seek=%d th*avg=%d skip=%"PRId64, b_seek, i_th*i_avg, i_skip ); } } @@ -914,19 +995,21 @@ static int AStreamSeekBlock( stream_t *s, int64_t i_pos ) } else { - /* Read enough data */ - while( p_sys->block.i_start + p_sys->block.i_size < i_pos ) + do { + /* Read and skip enough data */ if( AStreamRefillBlock( s ) ) return VLC_EGENERIC; while( p_sys->block.p_current && - p_sys->i_pos + p_sys->block.p_current->i_buffer < i_pos ) + p_sys->i_pos + p_sys->block.p_current->i_buffer - p_sys->block.i_offset < i_pos ) { - p_sys->i_pos += p_sys->block.p_current->i_buffer; + p_sys->i_pos += p_sys->block.p_current->i_buffer - p_sys->block.i_offset; p_sys->block.p_current = p_sys->block.p_current->p_next; + p_sys->block.i_offset = 0; } } + while( p_sys->block.i_start + p_sys->block.i_size < i_pos ); p_sys->block.i_offset = i_pos - p_sys->i_pos; p_sys->i_pos = i_pos; @@ -1025,15 +1108,20 @@ static int AStreamReadStream( stream_t *s, void *p_read, int i_read ) /* seek within this stream if possible, else use plain old read and discard */ stream_sys_t *p_sys = s->p_sys; access_t *p_access = p_sys->p_access; - bool b_aseek; - access2_Control( p_access, ACCESS_CAN_SEEK, &b_aseek ); - if( b_aseek ) - return AStreamSeekStream( s, p_sys->i_pos + i_read ) ? 0 : i_read; + + /* seeking after EOF is not what we want */ + if( !( p_access->info.b_eof ) ) + { + bool b_aseek; + access_Control( p_access, ACCESS_CAN_SEEK, &b_aseek ); + if( b_aseek ) + return AStreamSeekStream( s, p_sys->i_pos + i_read ) ? 0 : i_read; + } } #ifdef STREAM_DEBUG - msg_Dbg( s, "AStreamReadStream: %d pos="I64Fd" tk=%d start="I64Fd - " offset=%d end="I64Fd, + msg_Dbg( s, "AStreamReadStream: %d pos=%"PRId64" tk=%d start=%"PRId64 + " offset=%d end=%"PRId64, i_read, p_sys->i_pos, p_sys->stream.i_tk, tk->i_start, p_sys->stream.i_offset, tk->i_end ); #endif @@ -1087,8 +1175,8 @@ static int AStreamPeekStream( stream_t *s, const uint8_t **pp_peek, int i_read ) if( tk->i_start >= tk->i_end ) return 0; /* EOF */ #ifdef STREAM_DEBUG - msg_Dbg( s, "AStreamPeekStream: %d pos="I64Fd" tk=%d " - "start="I64Fd" offset=%d end="I64Fd, + msg_Dbg( s, "AStreamPeekStream: %d pos=%"PRId64" tk=%d " + "start=%"PRId64" offset=%d end=%"PRId64, i_read, p_sys->i_pos, p_sys->stream.i_tk, tk->i_start, p_sys->stream.i_offset, tk->i_end ); #endif @@ -1150,8 +1238,8 @@ static int AStreamSeekStream( stream_t *s, int64_t i_pos ) int i; #ifdef STREAM_DEBUG - msg_Dbg( s, "AStreamSeekStream: to "I64Fd" pos="I64Fd - " tk=%d start="I64Fd" offset=%d end="I64Fd, + msg_Dbg( s, "AStreamSeekStream: to %"PRId64" pos=%"PRId64 + " tk=%d start=%"PRId64" offset=%d end=%"PRId64, i_pos, p_sys->i_pos, p_sys->stream.i_tk, p_sys->stream.tk[p_sys->stream.i_tk].i_start, p_sys->stream.i_offset, @@ -1187,7 +1275,7 @@ static int AStreamSeekStream( stream_t *s, int64_t i_pos ) return VLC_SUCCESS; } - access2_Control( p_access, ACCESS_CAN_SEEK, &b_aseek ); + access_Control( p_access, ACCESS_CAN_SEEK, &b_aseek ); if( !b_aseek ) { /* We can't do nothing */ @@ -1206,8 +1294,8 @@ static int AStreamSeekStream( stream_t *s, int64_t i_pos ) if( i_pos >= tk->i_start && i_pos <= tk->i_end ) { #ifdef STREAM_DEBUG - msg_Dbg( s, "AStreamSeekStream: reusing %d start="I64Fd - " end="I64Fd, i, tk->i_start, tk->i_end ); + msg_Dbg( s, "AStreamSeekStream: reusing %d start=%"PRId64 + " end=%"PRId64, i, tk->i_start, tk->i_end ); #endif /* Seek at the end of the buffer */ @@ -1228,7 +1316,7 @@ static int AStreamSeekStream( stream_t *s, int64_t i_pos ) } } - access2_Control( p_access, ACCESS_CAN_SEEK, &b_afastseek ); + access_Control( p_access, ACCESS_CAN_SEEK, &b_afastseek ); /* FIXME compute seek cost (instead of static 'stupid' value) */ i_maxth = __MIN( p_sys->stream.i_read_size, STREAM_READ_ATONCE / 2 ); if( !b_afastseek ) @@ -1376,13 +1464,13 @@ static void AStreamPrebufferStream( stream_t *s ) /* Update stat */ p_sys->stat.i_bytes = tk->i_end - tk->i_start; p_sys->stat.i_read_time = i_date - i_start; - i_byterate = ( I64C(1000000) * p_sys->stat.i_bytes ) / + i_byterate = ( INT64_C(1000000) * p_sys->stat.i_bytes ) / (p_sys->stat.i_read_time+1); - msg_Dbg( s, "pre-buffering done "I64Fd" bytes in "I64Fd"s - " - I64Fd" kbytes/s", + msg_Dbg( s, "pre-buffering done %"PRId64" bytes in %"PRId64"s - " + "%"PRId64" kbytes/s", p_sys->stat.i_bytes, - p_sys->stat.i_read_time / I64C(1000000), + p_sys->stat.i_read_time / INT64_C(1000000), i_byterate / 1024 ); break; } @@ -1414,6 +1502,115 @@ static void AStreamPrebufferStream( stream_t *s ) } } +/**************************************************************************** + * Method 3: + ****************************************************************************/ + +static int AStreamReadImmediate( stream_t *s, void *p_read, int i_read ) +{ + stream_sys_t *p_sys = s->p_sys; + +#ifdef STREAM_DEBUG + msg_Dbg( s, "AStreamReadImmediate p_read=%p i_read=%d", + p_read, i_read ); +#endif + + /* First, check if we already have some data in the buffer, + * that we could copy directly */ + int i_copy = __MIN( stream_buffered_size( s ), i_read ); + if( i_copy ) + { +#ifdef STREAM_DEBUG + msg_Dbg( s, "AStreamReadImmediate: copy %d from %p", i_copy, stream_buffer( s ) ); +#endif + + assert( i_copy <= STREAM_CACHE_SIZE ); + + if( p_read ) + { + memcpy( p_read, stream_buffer( s ), i_copy ); + p_read = (uint8_t *)p_read + i_copy; + } + } + + /* Now that we've read our buffer we don't need its i_copy bytes */ + stream_buffer_empty( s, i_copy ); + + /* Now check if we have still to really read some data */ + int i_to_read = i_read - i_copy; + if( i_to_read ) + { + if( p_read ) + i_to_read = AReadStream( s, p_read, i_to_read ); + else + { + void * dummy = malloc(i_to_read); + i_to_read = AReadStream( s, dummy, i_to_read ); + free(dummy); + } + } + + p_sys->i_pos += i_to_read; + + return i_to_read + i_copy; +} + +static int AStreamPeekImmediate( stream_t *s, const uint8_t **pp_peek, int i_read ) +{ +#ifdef STREAM_DEBUG + msg_Dbg( s, "AStreamPeekImmediate: %d size=%"PRId64, + i_read, size_buffered_size( s ) ); +#endif + + /* Avoid problem, but that shouldn't happen */ + if( i_read > STREAM_CACHE_SIZE / 2 ) + i_read = STREAM_CACHE_SIZE / 2; + + int i_to_read = i_read - stream_buffered_size( s ); + if( i_to_read > 0 ) + { +#ifdef STREAM_DEBUG + msg_Dbg( s, "AStreamPeekImmediate: Reading %d", + i_to_read ); +#endif + i_to_read = AReadStream( s, stream_buffer( s ) + stream_buffered_size( s ), + i_to_read ); + + if( i_to_read > 0 ) + stream_buffer_fill( s, i_to_read ); + } + + *pp_peek = stream_buffer( s ); + + return __MIN(stream_buffered_size( s ), i_read); +} + +static int AStreamSeekImmediate( stream_t *s, int64_t i_pos ) +{ + stream_sys_t *p_sys = s->p_sys; + access_t *p_access = p_sys->p_access; + bool b_aseek; + +#ifdef STREAM_DEBUG + msg_Dbg( s, "AStreamSeekImmediate to %"PRId64" pos=%"PRId64 + i_pos, p_sys->i_pos ); +#endif + + access_Control( p_access, ACCESS_CAN_SEEK, &b_aseek ); + if( !b_aseek ) + { + /* We can't do nothing */ + msg_Dbg( s, "AStreamSeekImmediate: can't seek" ); + return VLC_EGENERIC; + } + + /* Just reset our buffer */ + stream_buffer_empty( s, stream_buffered_size( s ) ); + + if( ASeek( s, i_pos ) ) return VLC_EGENERIC; + + return VLC_SUCCESS; +} /**************************************************************************** * stream_ReadLine: @@ -1587,10 +1784,7 @@ char * stream_ReadLine( stream_t *s ) i_data = (psz_eol - (char *)p_data) + 1; p_line = realloc( p_line, i_line + i_data + s->i_char_width ); /* add \0 */ if( !p_line ) - { - msg_Err( s, "Out of memory when reallocating p_line" ); goto error; - } i_data = stream_Read( s, &p_line[i_line], i_data ); if( i_data <= 0 ) break; /* Hmmm */ i_line += i_data - s->i_char_width; /* skip \n */; @@ -1603,10 +1797,7 @@ char * stream_ReadLine( stream_t *s ) /* Read data (+1 for easy \0 append) */ p_line = realloc( p_line, i_line + STREAM_PROBE_LINE + s->i_char_width ); if( !p_line ) - { - msg_Err( s, "Out of memory when reallocating p_line" ); goto error; - } i_data = stream_Read( s, &p_line[i_line], STREAM_PROBE_LINE ); if( i_data <= 0 ) break; /* Hmmm */ i_line += i_data; @@ -1631,10 +1822,7 @@ char * stream_ReadLine( stream_t *s ) /* iconv */ psz_new_line = malloc( i_line ); if( psz_new_line == NULL ) - { - msg_Err( s, "Out of memory when allocating psz_new_line" ); goto error; - } i_in = i_out = (size_t)i_line; p_in = p_line; p_out = psz_new_line; @@ -1685,6 +1873,8 @@ static int AReadStream( stream_t *s, void *p_read, int i_read ) if( !p_sys->i_list ) { i_read = p_access->pf_read( p_access, p_read, i_read ); + if( p_access->b_die ) + vlc_object_kill( s ); if( p_input ) { vlc_mutex_lock( &p_input->p->counters.counters_lock ); @@ -1700,6 +1890,8 @@ static int AReadStream( stream_t *s, void *p_read, int i_read ) i_read = p_sys->p_list_access->pf_read( p_sys->p_list_access, p_read, i_read ); + if( p_access->b_die ) + vlc_object_kill( s ); /* If we reached an EOF then switch to the next stream in the list */ if( i_read == 0 && p_sys->i_list_index + 1 < p_sys->i_list ) @@ -1709,12 +1901,12 @@ static int AReadStream( stream_t *s, void *p_read, int i_read ) msg_Dbg( s, "opening input `%s'", psz_name ); - p_list_access = access2_New( s, p_access->psz_access, "", psz_name ); + p_list_access = access_New( s, p_access->psz_access, "", psz_name ); if( !p_list_access ) return 0; if( p_sys->p_list_access != p_access ) - access2_Delete( p_sys->p_list_access ); + access_Delete( p_sys->p_list_access ); p_sys->p_list_access = p_list_access; @@ -1751,8 +1943,10 @@ static block_t *AReadBlock( stream_t *s, bool *pb_eof ) if( !p_sys->i_list ) { p_block = p_access->pf_block( p_access ); + if( p_access->b_die ) + vlc_object_kill( s ); if( pb_eof ) *pb_eof = p_access->info.b_eof; - if( p_input && p_block && p_access->p_libvlc->b_stats ) + if( p_input && p_block && libvlc_stats (p_access) ) { vlc_mutex_lock( &p_input->p->counters.counters_lock ); stats_UpdateInteger( s, p_input->p->counters.p_read_bytes, @@ -1766,6 +1960,8 @@ static block_t *AReadBlock( stream_t *s, bool *pb_eof ) } p_block = p_sys->p_list_access->pf_block( p_access ); + if( p_access->b_die ) + vlc_object_kill( s ); b_eof = p_sys->p_list_access->info.b_eof; if( pb_eof ) *pb_eof = b_eof; @@ -1777,12 +1973,12 @@ static block_t *AReadBlock( stream_t *s, bool *pb_eof ) msg_Dbg( s, "opening input `%s'", psz_name ); - p_list_access = access2_New( s, p_access->psz_access, "", psz_name ); + p_list_access = access_New( s, p_access->psz_access, "", psz_name ); if( !p_list_access ) return 0; if( p_sys->p_list_access != p_access ) - access2_Delete( p_sys->p_list_access ); + access_Delete( p_sys->p_list_access ); p_sys->p_list_access = p_list_access; @@ -1832,7 +2028,7 @@ static int ASeek( stream_t *s, int64_t i_pos ) if( i != p_sys->i_list_index && i != 0 ) { p_list_access = - access2_New( s, p_access->psz_access, "", psz_name ); + access_New( s, p_access->psz_access, "", psz_name ); } else if( i != p_sys->i_list_index ) { @@ -1842,7 +2038,7 @@ static int ASeek( stream_t *s, int64_t i_pos ) if( p_list_access ) { if( p_sys->p_list_access != p_access ) - access2_Delete( p_sys->p_list_access ); + access_Delete( p_sys->p_list_access ); p_sys->p_list_access = p_list_access; }