Commit 4df5d3b0 authored by Felix Paul Kühne's avatar Felix Paul Kühne Committed by Jean-Baptiste Kempf

qtcapture: timing and speed improvements

it respects the 'live-caching' setting now, is vastly more responsive and even got correct timing.

Thanks to emi and Luca for the ideas and help
(cherry picked from commit e0745f6002c5d0b20a552d98ab3e385798e68815)
Signed-off-by: default avatarJean-Baptiste Kempf <jb@videolan.org>
parent d70a28c3
...@@ -79,6 +79,7 @@ vlc_module_end () ...@@ -79,6 +79,7 @@ vlc_module_end ()
CVImageBufferRef currentImageBuffer; CVImageBufferRef currentImageBuffer;
mtime_t currentPts; mtime_t currentPts;
mtime_t previousPts; mtime_t previousPts;
long timeScale;
} }
- (id)init; - (id)init;
- (void)outputVideoFrame:(CVImageBufferRef)videoFrame withSampleBuffer:(QTSampleBuffer *)sampleBuffer fromConnection:(QTCaptureConnection *)connection; - (void)outputVideoFrame:(CVImageBufferRef)videoFrame withSampleBuffer:(QTSampleBuffer *)sampleBuffer fromConnection:(QTCaptureConnection *)connection;
...@@ -94,6 +95,7 @@ vlc_module_end () ...@@ -94,6 +95,7 @@ vlc_module_end ()
currentImageBuffer = nil; currentImageBuffer = nil;
currentPts = 0; currentPts = 0;
previousPts = 0; previousPts = 0;
timeScale = 0;
} }
return self; return self;
} }
...@@ -107,6 +109,11 @@ vlc_module_end () ...@@ -107,6 +109,11 @@ vlc_module_end ()
[super dealloc]; [super dealloc];
} }
- (long)timeScale
{
return timeScale;
}
- (void)outputVideoFrame:(CVImageBufferRef)videoFrame withSampleBuffer:(QTSampleBuffer *)sampleBuffer fromConnection:(QTCaptureConnection *)connection - (void)outputVideoFrame:(CVImageBufferRef)videoFrame withSampleBuffer:(QTSampleBuffer *)sampleBuffer fromConnection:(QTCaptureConnection *)connection
{ {
// Store the latest frame // Store the latest frame
...@@ -119,7 +126,9 @@ vlc_module_end () ...@@ -119,7 +126,9 @@ vlc_module_end ()
{ {
imageBufferToRelease = currentImageBuffer; imageBufferToRelease = currentImageBuffer;
currentImageBuffer = videoFrame; currentImageBuffer = videoFrame;
currentPts = (mtime_t)(1000000L / [sampleBuffer presentationTime].timeScale * [sampleBuffer presentationTime].timeValue); QTTime timeStamp = [sampleBuffer presentationTime];
timeScale = timeStamp.timeScale;
currentPts = (mtime_t)(1000000L / timeScale * timeStamp.timeValue);
/* Try to use hosttime of the sample if available, because iSight Pts seems broken */ /* Try to use hosttime of the sample if available, because iSight Pts seems broken */
NSNumber *hosttime = (NSNumber *)[sampleBuffer attributeForKey:QTSampleBufferHostTimeAttribute]; NSNumber *hosttime = (NSNumber *)[sampleBuffer attributeForKey:QTSampleBufferHostTimeAttribute];
...@@ -170,6 +179,8 @@ struct demux_sys_t { ...@@ -170,6 +179,8 @@ struct demux_sys_t {
VLCDecompressedVideoOutput * output; VLCDecompressedVideoOutput * output;
int height, width; int height, width;
es_out_id_t * p_es_video; es_out_id_t * p_es_video;
BOOL b_es_setup;
es_format_t fmt;
}; };
...@@ -206,7 +217,6 @@ static int Open( vlc_object_t *p_this ) ...@@ -206,7 +217,6 @@ static int Open( vlc_object_t *p_this )
{ {
demux_t *p_demux = (demux_t*)p_this; demux_t *p_demux = (demux_t*)p_this;
demux_sys_t *p_sys = NULL; demux_sys_t *p_sys = NULL;
es_format_t fmt;
int i; int i;
int i_width; int i_width;
int i_height; int i_height;
...@@ -256,7 +266,7 @@ static int Open( vlc_object_t *p_this ) ...@@ -256,7 +266,7 @@ static int Open( vlc_object_t *p_this )
} }
} }
memset( &fmt, 0, sizeof( es_format_t ) ); memset( &p_sys->fmt, 0, sizeof( es_format_t ) );
QTCaptureDeviceInput * input = nil; QTCaptureDeviceInput * input = nil;
NSError *o_returnedError; NSError *o_returnedError;
...@@ -318,7 +328,7 @@ static int Open( vlc_object_t *p_this ) ...@@ -318,7 +328,7 @@ static int Open( vlc_object_t *p_this )
int chroma = VLC_CODEC_UYVY; int chroma = VLC_CODEC_UYVY;
/* Now we can init */ /* Now we can init */
es_format_Init( &fmt, VIDEO_ES, chroma ); es_format_Init( &p_sys->fmt, VIDEO_ES, chroma );
NSSize encoded_size = [[camera_format attributeForKey:QTFormatDescriptionVideoEncodedPixelsSizeAttribute] sizeValue]; NSSize encoded_size = [[camera_format attributeForKey:QTFormatDescriptionVideoEncodedPixelsSizeAttribute] sizeValue];
NSSize display_size = [[camera_format attributeForKey:QTFormatDescriptionVideoCleanApertureDisplaySizeAttribute] sizeValue]; NSSize display_size = [[camera_format attributeForKey:QTFormatDescriptionVideoCleanApertureDisplaySizeAttribute] sizeValue];
...@@ -329,12 +339,13 @@ static int Open( vlc_object_t *p_this ) ...@@ -329,12 +339,13 @@ static int Open( vlc_object_t *p_this )
par_size.height = display_size.height = encoded_size.height par_size.height = display_size.height = encoded_size.height
= var_InheritInteger (p_this, "qtcapture-height"); = var_InheritInteger (p_this, "qtcapture-height");
fmt.video.i_width = p_sys->width = encoded_size.width; p_sys->fmt.video.i_width = p_sys->width = encoded_size.width;
fmt.video.i_height = p_sys->height = encoded_size.height; p_sys->fmt.video.i_height = p_sys->height = encoded_size.height;
p_sys->fmt.video.i_frame_rate = 25.0; // cave: check with setMinimumVideoFrameInterval (see below)
if( par_size.width != encoded_size.width ) if( par_size.width != encoded_size.width )
{ {
fmt.video.i_sar_num = (int64_t)encoded_size.height * par_size.width / encoded_size.width; p_sys->fmt.video.i_sar_num = (int64_t)encoded_size.height * par_size.width / encoded_size.width;
fmt.video.i_sar_den = encoded_size.width; p_sys->fmt.video.i_sar_den = encoded_size.width;
} }
msg_Dbg(p_demux, "encoded_size %i %i", (int)encoded_size.width, (int)encoded_size.height ); msg_Dbg(p_demux, "encoded_size %i %i", (int)encoded_size.width, (int)encoded_size.height );
...@@ -347,6 +358,8 @@ static int Open( vlc_object_t *p_this ) ...@@ -347,6 +358,8 @@ static int Open( vlc_object_t *p_this )
[NSNumber numberWithInt: p_sys->width], kCVPixelBufferWidthKey, [NSNumber numberWithInt: p_sys->width], kCVPixelBufferWidthKey,
[NSNumber numberWithBool:YES], (id)kCVPixelBufferOpenGLCompatibilityKey, [NSNumber numberWithBool:YES], (id)kCVPixelBufferOpenGLCompatibilityKey,
nil]]; nil]];
[p_sys->output setAutomaticallyDropsLateVideoFrames:YES];
[p_sys->output setMinimumVideoFrameInterval: (1/25)]; // 25 fps
p_sys->session = [[QTCaptureSession alloc] init]; p_sys->session = [[QTCaptureSession alloc] init];
...@@ -366,11 +379,6 @@ static int Open( vlc_object_t *p_this ) ...@@ -366,11 +379,6 @@ static int Open( vlc_object_t *p_this )
[p_sys->session startRunning]; [p_sys->session startRunning];
msg_Dbg( p_demux, "added new video es %4.4s %dx%d",
(char*)&fmt.i_codec, fmt.video.i_width, fmt.video.i_height );
p_sys->p_es_video = es_out_Add( p_demux->out, &fmt );
[input release]; [input release];
[pool release]; [pool release];
...@@ -421,8 +429,7 @@ static int Demux( demux_t *p_demux ) ...@@ -421,8 +429,7 @@ static int Demux( demux_t *p_demux )
demux_sys_t *p_sys = p_demux->p_sys; demux_sys_t *p_sys = p_demux->p_sys;
block_t *p_block; block_t *p_block;
p_block = block_New( p_demux, p_sys->width * p_block = block_New( p_demux, p_sys->width * p_sys->height * 2 /* FIXME */ );
p_sys->height * 2 /* FIXME */ );
if( !p_block ) if( !p_block )
{ {
msg_Err( p_demux, "cannot get block" ); msg_Err( p_demux, "cannot get block" );
...@@ -444,6 +451,14 @@ static int Demux( demux_t *p_demux ) ...@@ -444,6 +451,14 @@ static int Demux( demux_t *p_demux )
msleep( 10000 ); msleep( 10000 );
return 1; return 1;
} }
else if( !p_sys->b_es_setup )
{
p_sys->fmt.video.i_frame_rate_base = [p_sys->output timeScale];
msg_Dbg( p_demux, "using frame rate base: %i", p_sys->fmt.video.i_frame_rate_base );
p_sys->p_es_video = es_out_Add( p_demux->out, &p_sys->fmt );
msg_Dbg( p_demux, "added new video es %4.4s %dx%d", (char*)&p_sys->fmt.i_codec, p_sys->fmt.video.i_width, p_sys->fmt.video.i_height );
p_sys->b_es_setup = YES;
}
es_out_Control( p_demux->out, ES_OUT_SET_PCR, p_block->i_pts ); es_out_Control( p_demux->out, ES_OUT_SET_PCR, p_block->i_pts );
es_out_Send( p_demux->out, p_sys->p_es_video, p_block ); es_out_Send( p_demux->out, p_sys->p_es_video, p_block );
...@@ -473,7 +488,12 @@ static int Control( demux_t *p_demux, int i_query, va_list args ) ...@@ -473,7 +488,12 @@ static int Control( demux_t *p_demux, int i_query, va_list args )
case DEMUX_GET_PTS_DELAY: case DEMUX_GET_PTS_DELAY:
pi64 = (int64_t*)va_arg( args, int64_t * ); pi64 = (int64_t*)va_arg( args, int64_t * );
*pi64 = (int64_t)DEFAULT_PTS_DELAY; *pi64 = INT64_C(1000) * var_InheritInteger( p_demux, "live-caching" );
return VLC_SUCCESS;
case DEMUX_GET_TIME:
pi64 = (int64_t*)va_arg( args, int64_t * );
*pi64 = mdate();
return VLC_SUCCESS; return VLC_SUCCESS;
default: default:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment