typedef enum stream_read_method_t
{
- Immediate,
- Block,
- Stream
+ STREAM_METHOD_IMMEDIATE,
+ STREAM_METHOD_BLOCK,
+ STREAM_METHOD_STREAM
} stream_read_method_t;
struct stream_sys_t
{
bool b_active;
- FILE *f; /* TODO it could be replaced by access_output_t one day */
+ FILE *f; /* TODO it could be replaced by access_output_t one day */
+ bool b_error;
} record;
};
/* Common field */
p_sys->p_access = p_access;
if( p_access->pf_block )
- p_sys->method = Block;
- else if (var_CreateGetBool( s, "use-stream-immediate"))
- p_sys->method = Immediate;
+ p_sys->method = STREAM_METHOD_BLOCK;
+ else if( var_CreateGetBool( s, "use-stream-immediate" ) )
+ p_sys->method = STREAM_METHOD_IMMEDIATE;
else
- p_sys->method = Stream;
+ p_sys->method = STREAM_METHOD_STREAM;
p_sys->record.b_active = false;
p_sys->i_peek = 0;
p_sys->p_peek = NULL;
- if( p_sys->method == Block )
+ if( p_sys->method == STREAM_METHOD_BLOCK )
{
msg_Dbg( s, "Using AStream*Block" );
s->pf_read = AStreamReadBlock;
goto error;
}
}
- else if (p_sys->method == Immediate)
+ else if( p_sys->method == STREAM_METHOD_IMMEDIATE )
{
msg_Dbg( s, "Using AStream*Immediate" );
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 );
+ 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 )
{
goto error;
}
}
- else /* ( p_sys->method == Stream ) */
+ else
{
int i;
+ assert( p_sys->method == STREAM_METHOD_STREAM );
+
msg_Dbg( s, "Using AStream*Stream" );
s->pf_read = AStreamReadStream;
return s;
error:
- if( p_sys->method == Block )
+ if( p_sys->method == STREAM_METHOD_BLOCK )
{
/* Nothing yet */
}
if( p_sys->record.b_active )
ARecordSetState( s, false, NULL );
- if( p_sys->method == Block )
+ if( p_sys->method == STREAM_METHOD_BLOCK )
block_ChainRelease( p_sys->block.p_first );
- else if ( p_sys->method == Immediate )
+ else if( p_sys->method == STREAM_METHOD_IMMEDIATE )
free( p_sys->immediate.p_buffer );
else
free( p_sys->stream.p_buffer );
p_sys->i_pos = p_sys->p_access->info.i_pos;
- if( p_sys->method == Block )
+ if( p_sys->method == STREAM_METHOD_BLOCK )
{
block_ChainRelease( p_sys->block.p_first );
/* Do the prebuffering */
AStreamPrebufferBlock( s );
}
- else if( p_sys->method == Immediate )
+ else if( p_sys->method == STREAM_METHOD_IMMEDIATE )
{
stream_buffer_empty( s, stream_buffered_size( s ) );
}
- else /* ( p_sys->method == Stream ) */
+ else
{
int i;
+ assert( p_sys->method == STREAM_METHOD_STREAM );
+
/* Setup our tracks */
p_sys->stream.i_offset = 0;
p_sys->stream.i_tk = 0;
case STREAM_SET_POSITION:
i_64 = (int64_t)va_arg( args, int64_t );
- if( p_sys->method == Block )
+ switch( p_sys->method )
+ {
+ case STREAM_METHOD_BLOCK:
return AStreamSeekBlock( s, i_64 );
- else if( p_sys->method == Immediate )
+ case STREAM_METHOD_IMMEDIATE:
return AStreamSeekImmediate( s, i_64 );
- else /* ( p_sys->method == Stream ) */
+ case STREAM_METHOD_STREAM:
return AStreamSeekStream( s, i_64 );
+ default:
+ assert(0);
+ return VLC_EGENERIC;
+ }
case STREAM_GET_MTU:
return VLC_EGENERIC;
{
closedir( path );
- const char *psz_prefix = "vlc-record-%Y-%m-%d-%H:%M:%S-$p"; // TODO allow conf ?
char *psz_tmp = str_format( s, psz_prefix );
if( !psz_tmp )
return NULL;
{
stream_sys_t *p_sys = s->p_sys;
- DIR *path;
char *psz_file;
FILE *f;
/* */
p_sys->record.f = f;
p_sys->record.b_active = true;
+ p_sys->record.b_error = false;
return VLC_SUCCESS;
}
static int ARecordStop( stream_t *s )
assert( p_sys->record.b_active );
- if( i_buffer )
- fwrite( p_buffer, 1, i_buffer, p_sys->record.f );
+ if( i_buffer > 0 )
+ {
+ const bool b_previous_error = p_sys->record.b_error;
+ const size_t i_written = fwrite( p_buffer, 1, i_buffer, p_sys->record.f );
+
+ p_sys->record.b_error = i_written != i_buffer;
+
+ /* TODO maybe a intf_UserError or something like that ? */
+ if( p_sys->record.b_error && !b_previous_error )
+ msg_Err( s, "Failed to record data (begin)" );
+ else if( !p_sys->record.b_error && b_previous_error )
+ msg_Err( s, "Failed to record data (end)" );
+ }
}
/****************************************************************************
{
stream_sys_t *p_sys = s->p_sys;
- uint8_t *p_data= (uint8_t*)p_read;
+ uint8_t *p_data = p_read;
+ uint8_t *p_record = p_data;
unsigned int i_data = 0;
/* It means EOF */
if( p_sys->block.p_current == NULL )
return 0;
- if( p_read == NULL )
+ if( p_sys->record.b_active && !p_data )
+ p_record = p_data = malloc( i_read );
+
+ if( p_data == NULL )
{
/* seek within this stream if possible, else use plain old read and discard */
stream_sys_t *p_sys = s->p_sys;
}
}
- if( p_sys->record.b_active && i_data > 0 )
- ARecordWrite( s, p_read, i_data );
+ if( p_sys->record.b_active )
+ {
+ if( i_data > 0 && p_record != NULL)
+ ARecordWrite( s, p_record, i_data );
+ if( !p_read )
+ free( p_record );
+ }
p_sys->i_pos += i_data;
return i_data;
stream_track_t *tk = &p_sys->stream.tk[p_sys->stream.i_tk];
uint8_t *p_data = (uint8_t *)p_read;
+ uint8_t *p_record = p_data;
unsigned int i_data = 0;
- if( tk->i_start >= tk->i_end ) return 0; /* EOF */
+ if( tk->i_start >= tk->i_end )
+ return 0; /* EOF */
- if( p_read == NULL )
+ if( p_sys->record.b_active && !p_data )
+ p_record = p_data = malloc( i_read );
+
+ if( p_data == NULL )
{
/* seek within this stream if possible, else use plain old read and discard */
stream_sys_t *p_sys = s->p_sys;
}
}
- if( p_sys->record.b_active && i_data > 0 )
- ARecordWrite( s, p_read, i_data );
+ if( p_sys->record.b_active )
+ {
+ if( i_data > 0 && p_record != NULL)
+ ARecordWrite( s, p_record, i_data );
+ if( !p_read )
+ free( p_record );
+ }
return i_data;
}
static int AStreamReadImmediate( stream_t *s, void *p_read, unsigned int i_read )
{
stream_sys_t *p_sys = s->p_sys;
+ uint8_t *p_data= (uint8_t*)p_read;
+ uint8_t *p_record = p_data;
+ unsigned int i_data;
#ifdef STREAM_DEBUG
msg_Dbg( s, "AStreamReadImmediate p_read=%p i_read=%d",
p_read, i_read );
#endif
+ if( p_sys->record.b_active && !p_data )
+ p_record = p_data = malloc( i_read );
+
/* 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 )
+ i_data = __MIN( stream_buffered_size( s ), i_read );
+ if( i_data > 0 )
{
#ifdef STREAM_DEBUG
- msg_Dbg( s, "AStreamReadImmediate: copy %d from %p", i_copy, stream_buffer( s ) );
+ msg_Dbg( s, "AStreamReadImmediate: copy %u from %p", i_data, stream_buffer( s ) );
#endif
- assert( i_copy <= STREAM_CACHE_SIZE );
+ assert( i_data <= STREAM_CACHE_SIZE );
- if( p_read )
+ if( p_data )
{
- memcpy( p_read, stream_buffer( s ), i_copy );
- p_read = (uint8_t *)p_read + i_copy;
+ memcpy( p_data, stream_buffer( s ), i_data );
+ p_data += i_data;
}
}
/* Now that we've read our buffer we don't need its i_copy bytes */
- stream_buffer_empty( s, i_copy );
+ stream_buffer_empty( s, i_data );
/* Now check if we have still to really read some data */
- int i_to_read = i_read - i_copy;
- if( i_to_read )
+ while( i_data < i_read )
{
- if( p_read )
- i_to_read = AReadStream( s, p_read, i_to_read );
- else
+ const unsigned int i_to_read = i_read - i_data;
+ int i_result;
+
+ if( p_data )
{
- void * dummy = malloc(i_to_read);
- i_to_read = AReadStream( s, dummy, i_to_read );
- free(dummy);
+ i_result = AReadStream( s, p_data, i_to_read );
}
- }
+ else
+ {
+ void *p_dummy = malloc( i_to_read );
+
+ if( !p_dummy )
+ break;
- if( p_sys->record.b_active && i_copy > 0 )
- ARecordWrite( s, p_read, i_copy );
+ i_result = AReadStream( s, p_dummy, i_to_read );
- p_sys->i_pos += i_to_read;
+ free( p_dummy );
+ }
+ if( i_result <= 0 )
+ break;
+
+ p_sys->i_pos += i_data;
+ if( p_data )
+ p_data += i_result;
+ i_data += i_result;
+ }
- return i_to_read + i_copy;
+ if( p_sys->record.b_active )
+ {
+ if( i_data > 0 && p_record != NULL)
+ ARecordWrite( s, p_record, i_data );
+ if( !p_read )
+ free( p_record );
+ }
+ return i_data;
}
static int AStreamPeekImmediate( stream_t *s, const uint8_t **pp_peek, unsigned int i_read )
i_read, size_buffered_size( s ) );
#endif
- /* Avoid problem, but that shouldn't happen */
+ /* Avoid problem, but that shouldn't happen
+ * FIXME yes it can */
if( i_read > STREAM_CACHE_SIZE / 2 )
i_read = STREAM_CACHE_SIZE / 2;
*pp_peek = stream_buffer( s );
- return __MIN(stream_buffered_size( s ), i_read);
+ return __MIN( stream_buffered_size( s ), i_read );
}
static int AStreamSeekImmediate( stream_t *s, int64_t i_pos )
/* Just reset our buffer */
stream_buffer_empty( s, stream_buffered_size( s ) );
- if( ASeek( s, i_pos ) ) return VLC_EGENERIC;
+ if( ASeek( s, i_pos ) )
+ return VLC_EGENERIC;
return VLC_SUCCESS;
}
va_list args;
int i_result;
- if ( s == NULL )
+ if( s == NULL )
return VLC_EGENERIC;
va_start( args, i_query );