Commit 45dd5c69 authored by Fabrice Bellard's avatar Fabrice Bellard

a/v sync support: added correct pts handling

Originally committed as revision 1229 to svn://svn.ffmpeg.org/ffmpeg/trunk
parent 8949367e
...@@ -220,21 +220,25 @@ static int audio_read_header(AVFormatContext *s1, AVFormatParameters *ap) ...@@ -220,21 +220,25 @@ static int audio_read_header(AVFormatContext *s1, AVFormatParameters *ap)
if (ret < 0) { if (ret < 0) {
av_free(st); av_free(st);
return -EIO; return -EIO;
} else {
/* take real parameters */
st->codec.codec_type = CODEC_TYPE_AUDIO;
st->codec.codec_id = s->codec_id;
st->codec.sample_rate = s->sample_rate;
st->codec.channels = s->channels;
return 0;
} }
/* take real parameters */
st->codec.codec_type = CODEC_TYPE_AUDIO;
st->codec.codec_id = s->codec_id;
st->codec.sample_rate = s->sample_rate;
st->codec.channels = s->channels;
av_set_pts_info(s1, 48, 1, 1000000); /* 48 bits pts in us */
return 0;
} }
static int audio_read_packet(AVFormatContext *s1, AVPacket *pkt) static int audio_read_packet(AVFormatContext *s1, AVPacket *pkt)
{ {
AudioData *s = s1->priv_data; AudioData *s = s1->priv_data;
int ret; int ret, bdelay;
int64_t cur_time;
struct audio_buf_info abufi;
if (av_new_packet(pkt, s->frame_size) < 0) if (av_new_packet(pkt, s->frame_size) < 0)
return -EIO; return -EIO;
for(;;) { for(;;) {
...@@ -252,6 +256,19 @@ static int audio_read_packet(AVFormatContext *s1, AVPacket *pkt) ...@@ -252,6 +256,19 @@ static int audio_read_packet(AVFormatContext *s1, AVPacket *pkt)
} }
} }
pkt->size = ret; pkt->size = ret;
/* compute pts of the start of the packet */
cur_time = av_gettime();
bdelay = ret;
if (ioctl(s->fd, SNDCTL_DSP_GETISPACE, &abufi) == 0) {
bdelay += abufi.bytes;
}
/* substract time represented by the number of bytes in the audio fifo */
cur_time -= (bdelay * 1000000LL) / (s->sample_rate * s->channels);
/* convert to wanted units */
pkt->pts = cur_time & ((1LL << 48) - 1);
if (s->flip_left && s->channels == 2) { if (s->flip_left && s->channels == 2) {
int i; int i;
short *p = (short *) pkt->data; short *p = (short *) pkt->data;
......
...@@ -240,6 +240,8 @@ static int grab_read_header(AVFormatContext *s1, AVFormatParameters *ap) ...@@ -240,6 +240,8 @@ static int grab_read_header(AVFormatContext *s1, AVFormatParameters *ap)
st->codec.height = height; st->codec.height = height;
st->codec.frame_rate = frame_rate; st->codec.frame_rate = frame_rate;
av_set_pts_info(s1, 48, 1, 1000000); /* 48 bits pts in us */
return 0; return 0;
fail: fail:
if (video_fd >= 0) if (video_fd >= 0)
...@@ -279,7 +281,6 @@ static int grab_read_packet(AVFormatContext *s1, AVPacket *pkt) ...@@ -279,7 +281,6 @@ static int grab_read_packet(AVFormatContext *s1, AVPacket *pkt)
VideoData *s = s1->priv_data; VideoData *s = s1->priv_data;
INT64 curtime, delay; INT64 curtime, delay;
struct timespec ts; struct timespec ts;
int first;
INT64 per_frame = (INT64_C(1000000) * FRAME_RATE_BASE) / s->frame_rate; INT64 per_frame = (INT64_C(1000000) * FRAME_RATE_BASE) / s->frame_rate;
int dropped = 0; int dropped = 0;
...@@ -287,7 +288,7 @@ static int grab_read_packet(AVFormatContext *s1, AVPacket *pkt) ...@@ -287,7 +288,7 @@ static int grab_read_packet(AVFormatContext *s1, AVPacket *pkt)
s->time_frame += per_frame; s->time_frame += per_frame;
/* wait based on the frame rate */ /* wait based on the frame rate */
for(first = 1;; first = 0) { for(;;) {
curtime = av_gettime(); curtime = av_gettime();
delay = s->time_frame - curtime; delay = s->time_frame - curtime;
if (delay <= 0) { if (delay <= 0) {
...@@ -306,6 +307,8 @@ static int grab_read_packet(AVFormatContext *s1, AVPacket *pkt) ...@@ -306,6 +307,8 @@ static int grab_read_packet(AVFormatContext *s1, AVPacket *pkt)
if (av_new_packet(pkt, s->frame_size) < 0) if (av_new_packet(pkt, s->frame_size) < 0)
return -EIO; return -EIO;
pkt->pts = curtime & ((1LL << 48) - 1);
if (dropped) if (dropped)
pkt->flags |= PKT_FLAG_DROPPED_FRAME; pkt->flags |= PKT_FLAG_DROPPED_FRAME;
......
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