basevideoencoder: Adjusting padding is not required for -bad libraries
authorSebastian Dröge <sebastian.droege@collabora.co.uk>
Tue, 29 Nov 2011 08:18:19 +0000 (09:18 +0100)
committerSebastian Dröge <sebastian.droege@collabora.co.uk>
Tue, 29 Nov 2011 11:18:19 +0000 (12:18 +0100)
gst-libs/gst/video/gstbasevideoencoder.c
gst-libs/gst/video/gstbasevideoencoder.h

index 15fba3ce4b329eecb6152bda297175f6abddf260..2c58463529f885b2e171095aed72f26922ed8bcd 100644 (file)
@@ -223,7 +223,7 @@ gst_base_video_encoder_init (GstBaseVideoEncoder * base_video_encoder,
   gst_pad_set_event_function (pad,
       GST_DEBUG_FUNCPTR (gst_base_video_encoder_src_event));
 
-  base_video_encoder->a.at_eos = FALSE;
+  base_video_encoder->at_eos = FALSE;
   base_video_encoder->headers = NULL;
 
   /* encoder is expected to do so */
@@ -472,7 +472,7 @@ gst_base_video_encoder_sink_eventfunc (GstBaseVideoEncoder * base_video_encoder,
       GstFlowReturn flow_ret;
 
       GST_BASE_VIDEO_CODEC_STREAM_LOCK (base_video_encoder);
-      base_video_encoder->a.at_eos = TRUE;
+      base_video_encoder->at_eos = TRUE;
 
       if (base_video_encoder_class->finish) {
         flow_ret = base_video_encoder_class->finish (base_video_encoder);
@@ -510,7 +510,7 @@ gst_base_video_encoder_sink_eventfunc (GstBaseVideoEncoder * base_video_encoder,
         break;
       }
 
-      base_video_encoder->a.at_eos = FALSE;
+      base_video_encoder->at_eos = FALSE;
 
       gst_segment_set_newsegment_full (&GST_BASE_VIDEO_CODEC
           (base_video_encoder)->segment, update, rate, applied_rate, format,
@@ -746,7 +746,7 @@ gst_base_video_encoder_chain (GstPad * pad, GstBuffer * buf)
       GST_TIME_ARGS (GST_BUFFER_TIMESTAMP (buf)),
       GST_TIME_ARGS (GST_BUFFER_DURATION (buf)));
 
-  if (base_video_encoder->a.at_eos) {
+  if (base_video_encoder->at_eos) {
     ret = GST_FLOW_UNEXPECTED;
     goto done;
   }
index 2de8af09126b3b42f21631b9bdb90b9a47c83a0f..df3d111845371534870ed833a3b041427554fab9 100644 (file)
@@ -90,6 +90,7 @@ struct _GstBaseVideoEncoder
   /* FIXME move to real private part ?
    * (and introduce a context ?) */
   gboolean          drained;
+  gboolean          at_eos;
 
   gint64            min_latency;
   gint64            max_latency;
@@ -101,13 +102,7 @@ struct _GstBaseVideoEncoder
   gboolean          force_keyframe_pending;
   gboolean          force_keyframe_headers;
 
-  union {
-    void *padding;
-    gboolean at_eos;
-  } a;
-
-  /* FIXME before moving to base */
-  void             *padding[GST_PADDING_LARGE-1];
+  void             *padding[GST_PADDING_LARGE];
 };
 
 /**