encoder: h264: disable NAL HRD parameters for now.
authorGwenole Beauchesne <gwenole.beauchesne@intel.com>
Wed, 22 Jan 2014 16:07:24 +0000 (17:07 +0100)
committerGwenole Beauchesne <gwenole.beauchesne@intel.com>
Wed, 22 Jan 2014 17:30:26 +0000 (18:30 +0100)
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

index c4421d929d0fc4c3ef0aba8c954977d764a78ff5..9b794551623792000f50b48f424bf9a7b1632906 100644 (file)
@@ -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 */