FaSTTUBe/GST-Tensordecoder-ov_ep
37
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

x264enc: handle possibly negative DTS provided by codec

Browse Source 
... by arranging for an offset such that DTS == PTS for keyframes,
which is expected elsewhere to go along with semantics of PTS and DTS.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=679443
This commit is contained in:
Mark Nauwelaerts 2012-09-07 17:38:18 +02:00
parent 67342c46b6
commit 839e75d5d1
2 changed files with 20 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
ext/x264/gstx264enc.c
View File

@ -1170,6 +1170,8 @@ gst_x264_enc_init_encoder (GstX264Enc * encoder)
} }
encoder->reconfig = FALSE; encoder->reconfig = FALSE;
/* good start, will be corrected if needed */
encoder->dts_offset = 0;
GST_OBJECT_UNLOCK (encoder); GST_OBJECT_UNLOCK (encoder);
@ -1699,11 +1701,24 @@ gst_x264_enc_encode_frame (GstX264Enc * encoder, x264_picture_t * pic_in,
"output: dts %" G_GINT64_FORMAT " pts %" G_GINT64_FORMAT, "output: dts %" G_GINT64_FORMAT " pts %" G_GINT64_FORMAT,
(gint64) pic_out.i_dts, (gint64) pic_out.i_pts); (gint64) pic_out.i_dts, (gint64) pic_out.i_pts);
if (pic_out.i_dts < 0) /* we want to know if x264 is messing around with this */
g_assert (frame->pts == pic_out.i_pts);
if (pic_out.b_keyframe) {
/* expect dts == pts, and also positive ts,
* so arrange for an offset if needed */
if (pic_out.i_dts + encoder->dts_offset != pic_out.i_pts) {
encoder->dts_offset = pic_out.i_pts - pic_out.i_dts;
GST_DEBUG_OBJECT (encoder, "determined dts offset %" G_GINT64_FORMAT,
encoder->dts_offset);
}
}
frame->dts = pic_out.i_dts + encoder->dts_offset;
/* should be ok now, surprise if not */
if (frame->dts < 0) {
GST_WARNING_OBJECT (encoder, "negative dts after offset compensation");
frame->dts = GST_CLOCK_TIME_NONE; frame->dts = GST_CLOCK_TIME_NONE;
else }
frame->dts = pic_out.i_dts;
frame->pts = pic_out.i_pts;
if (pic_out.b_keyframe) { if (pic_out.b_keyframe) {
GST_INFO ("Output keyframe"); GST_INFO ("Output keyframe");

1
ext/x264/gstx264enc.h
View File

@ -51,6 +51,7 @@ struct _GstX264Enc
x264_t *x264enc; x264_t *x264enc;
x264_param_t x264param; x264_param_t x264param;
gint current_byte_stream; gint current_byte_stream;
GstClockTime dts_offset;
/* List of frame/buffer mapping structs for /* List of frame/buffer mapping structs for
* pending frames */ * pending frames */