From deff9c775c477ff80112ae11d080dea9a7f43c9d Mon Sep 17 00:00:00 2001 From: Gwenole Beauchesne Date: Wed, 22 Jan 2014 17:07:24 +0100 Subject: [PATCH] encoder: h264: disable NAL HRD parameters for now. Don't emit NAL HRD parameters for now in the SPS headers because the SEI buffering_period() and picture_timing() messages are not handled yet. Some additional changes are necessary to get it right. https://bugzilla.gnome.org/show_bug.cgi?id=722734 --- gst-libs/gst/vaapi/gstvaapiencoder_h264.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/gst-libs/gst/vaapi/gstvaapiencoder_h264.c b/gst-libs/gst/vaapi/gstvaapiencoder_h264.c index c4421d929d..9b79455162 100644 --- a/gst-libs/gst/vaapi/gstvaapiencoder_h264.c +++ b/gst-libs/gst/vaapi/gstvaapiencoder_h264.c @@ -417,9 +417,9 @@ bs_write_sps (GstBitWriter * bs, WRITE_UINT32 (bs, 1, 1); /* fixed_frame_rate_flag */ } - nal_hrd_parameters_present_flag = - (seq_param->bits_per_second > 0 ? TRUE : FALSE); /* nal_hrd_parameters_present_flag */ + nal_hrd_parameters_present_flag = seq_param->bits_per_second > 0; + nal_hrd_parameters_present_flag = FALSE; /* XXX: disabled for now */ WRITE_UINT32 (bs, nal_hrd_parameters_present_flag, 1); if (nal_hrd_parameters_present_flag) { /* hrd_parameters */ @@ -445,8 +445,10 @@ bs_write_sps (GstBitWriter * bs, /* time_offset_length */ WRITE_UINT32 (bs, 23, 5); } + /* vcl_hrd_parameters_present_flag */ WRITE_UINT32 (bs, 0, 1); + if (nal_hrd_parameters_present_flag || 0 /*vcl_hrd_parameters_present_flag */ ) { /* low_delay_hrd_flag */