ext/ffmpeg/gstffmpegdec.c: Add some more debug, fix for B-frame containing XviD movies.

Original commit message from CVS:
* ext/ffmpeg/gstffmpegdec.c: (gst_ffmpegdec_open),
(gst_ffmpegdec_connect), (gst_ffmpegdec_negotiate),
(gst_ffmpegdec_frame), (gst_ffmpegdec_handle_event):
Add some more debug, fix for B-frame containing XviD movies.
This commit is contained in:
Ronald S. Bultje 2005-01-19 15:34:14 +00:00
parent 0ee46f8b0f
commit ef38fec2d0
2 changed files with 31 additions and 4 deletions

View file

@ -1,3 +1,10 @@
2005-01-19 Ronald S. Bultje <rbultje@ronald.bitfreak.net>
* ext/ffmpeg/gstffmpegdec.c: (gst_ffmpegdec_open),
(gst_ffmpegdec_connect), (gst_ffmpegdec_negotiate),
(gst_ffmpegdec_frame), (gst_ffmpegdec_handle_event):
Add some more debug, fix for B-frame containing XviD movies.
2005-01-19 Ronald S. Bultje <rbultje@ronald.bitfreak.net> 2005-01-19 Ronald S. Bultje <rbultje@ronald.bitfreak.net>
* ext/ffmpeg/gstffmpegdec.c: (gst_ffmpegdec_handle_event): * ext/ffmpeg/gstffmpegdec.c: (gst_ffmpegdec_handle_event):

View file

@ -330,13 +330,16 @@ gst_ffmpegdec_open (GstFFMpegDec *ffmpegdec)
return FALSE; return FALSE;
} }
GST_LOG ("Opened ffmpeg codec %s", oclass->in_plugin->name);
/* open a parser if we can - exclude mpeg4, because it is already /* open a parser if we can - exclude mpeg4, because it is already
* framed (divx), mp3 because it doesn't work (?) and mjpeg because * framed (divx), mp3 because it doesn't work (?) and mjpeg because
* of $(see mpeg4)... */ * of $(see mpeg4)... */
if (oclass->in_plugin->id != CODEC_ID_MPEG4 && if (oclass->in_plugin->id != CODEC_ID_MPEG4 &&
oclass->in_plugin->id != CODEC_ID_MJPEG && oclass->in_plugin->id != CODEC_ID_MJPEG &&
oclass->in_plugin->id != CODEC_ID_MP3) oclass->in_plugin->id != CODEC_ID_MP3) {
ffmpegdec->pctx = av_parser_init (oclass->in_plugin->id); ffmpegdec->pctx = av_parser_init (oclass->in_plugin->id);
}
switch (oclass->in_plugin->type) { switch (oclass->in_plugin->type) {
case CODEC_TYPE_VIDEO: case CODEC_TYPE_VIDEO:
@ -394,9 +397,10 @@ gst_ffmpegdec_connect (GstPad * pad, const GstCaps * caps)
/* we dont send complete frames - FIXME: we need a 'framed' property /* we dont send complete frames - FIXME: we need a 'framed' property
* in caps */ * in caps */
if (oclass->in_plugin->capabilities & CODEC_CAP_TRUNCATED && if (oclass->in_plugin->capabilities & CODEC_CAP_TRUNCATED &&
(ffmpegdec->context->codec_id == CODEC_ID_MPEG1VIDEO || (oclass->in_plugin->id == CODEC_ID_MPEG1VIDEO ||
ffmpegdec->context->codec_id == CODEC_ID_MPEG2VIDEO)) oclass->in_plugin->id == CODEC_ID_MPEG2VIDEO)) {
ffmpegdec->context->flags |= CODEC_FLAG_TRUNCATED; ffmpegdec->context->flags |= CODEC_FLAG_TRUNCATED;
}
/* do *not* draw edges */ /* do *not* draw edges */
ffmpegdec->context->flags |= CODEC_FLAG_EMU_EDGE; ffmpegdec->context->flags |= CODEC_FLAG_EMU_EDGE;
@ -486,12 +490,26 @@ gst_ffmpegdec_negotiate (GstFFMpegDec * ffmpegdec)
ffmpegdec->format.video.fps_base == ffmpegdec->format.video.fps_base ==
ffmpegdec->context->frame_rate_base) ffmpegdec->context->frame_rate_base)
return TRUE; return TRUE;
GST_DEBUG ("Renegotiating video from %dx%d@%d/%dfps to %dx%d@%d/%dfps",
ffmpegdec->format.video.width, ffmpegdec->format.video.height,
ffmpegdec->format.video.fps, ffmpegdec->format.video.fps_base,
ffmpegdec->context->width, ffmpegdec->context->height,
ffmpegdec->context->frame_rate, ffmpegdec->context->frame_rate_base);
ffmpegdec->format.video.width = ffmpegdec->context->width;
ffmpegdec->format.video.height = ffmpegdec->context->height;
ffmpegdec->format.video.fps = ffmpegdec->context->frame_rate;
ffmpegdec->format.video.fps_base = ffmpegdec->context->frame_rate_base;
break; break;
case CODEC_TYPE_AUDIO: case CODEC_TYPE_AUDIO:
if (ffmpegdec->format.audio.samplerate == if (ffmpegdec->format.audio.samplerate ==
ffmpegdec->context->sample_rate && ffmpegdec->context->sample_rate &&
ffmpegdec->format.audio.channels == ffmpegdec->context->channels) ffmpegdec->format.audio.channels == ffmpegdec->context->channels)
return TRUE; return TRUE;
GST_DEBUG ("Renegotiating audio from %dHz@%dchannels to %dHz@%dchannels",
ffmpegdec->format.audio.samplerate, ffmpegdec->format.audio.channels,
ffmpegdec->context->sample_rate, ffmpegdec->context->channels);
ffmpegdec->format.audio.samplerate = ffmpegdec->context->sample_rate;
ffmpegdec->format.audio.channels == ffmpegdec->context->channels;
break; break;
default: default:
break; break;
@ -635,7 +653,7 @@ gst_ffmpegdec_frame (GstFFMpegDec * ffmpegdec,
*got_data = 0; *got_data = 0;
return 0; return 0;
} else { } else {
*got_data = 1; *got_data = have_data ? 1 : 0;
} }
if (have_data) { if (have_data) {
@ -658,6 +676,8 @@ gst_ffmpegdec_frame (GstFFMpegDec * ffmpegdec,
static void static void
gst_ffmpegdec_handle_event (GstFFMpegDec * ffmpegdec, GstEvent * event) gst_ffmpegdec_handle_event (GstFFMpegDec * ffmpegdec, GstEvent * event)
{ {
GST_DEBUG ("Handling event of type %d", GST_EVENT_TYPE (event));
switch (GST_EVENT_TYPE (event)) { switch (GST_EVENT_TYPE (event)) {
case GST_EVENT_FLUSH: case GST_EVENT_FLUSH:
if (ffmpegdec->opened) { if (ffmpegdec->opened) {