subenc: port to 1.0

configure.ac

@@ -318,7 +318,7 @@ GST_PLUGINS_NONPORTED=" androidmedia aiff \
  kate librfb \
  mpegpsmux mve mxf mythtv nsf nuvdemux \
  patchdetect real \
- sdi subenc stereo tta videofilters \
+ sdi stereo tta videofilters \
  videomeasure videosignal vmnc \
  decklink fbdev linsys vcd \
  apexsink cdaudio cog dc1394 dirac directfb \

gst/subenc/gstsrtenc.c

@@ -20,10 +20,10 @@
 #ifdef HAVE_CONFIG_H
 #include "config.h"
 #endif
-#include "string.h"
+
+#include <string.h>
 
 #include "gstsrtenc.h"
-#include <gst/controller/gstcontroller.h>
 
 GST_DEBUG_CATEGORY_STATIC (srtenc_debug);
 #define GST_CAT_DEFAULT srtenc_debug
@@ -43,21 +43,23 @@ static GstStaticPadTemplate src_template = GST_STATIC_PAD_TEMPLATE ("src",
 static GstStaticPadTemplate sink_template = GST_STATIC_PAD_TEMPLATE ("sink",
     GST_PAD_SINK,
     GST_PAD_ALWAYS,
-    GST_STATIC_CAPS ("text/plain; text/x-pango-markup"));
+    GST_STATIC_CAPS ("text/x-raw, format = { pango-markup, utf8 }"));
 
-static GstFlowReturn gst_srt_enc_chain (GstPad * pad, GstBuffer * buf);
+static GstFlowReturn gst_srt_enc_chain (GstPad * pad, GstObject * parent,
-static gchar *gst_srt_enc_timeconvertion (GstSrtEnc * srtenc, GstBuffer * buf);
+    GstBuffer * buf);
-static gchar *gst_srt_enc_timestamp_to_string (GstClockTime timestamp);
+static gboolean gst_srt_enc_event (GstPad * pad, GstObject * parent,
+    GstEvent * event);
 static void gst_srt_enc_get_property (GObject * object, guint prop_id,
     GValue * value, GParamSpec * pspec);
 static void gst_srt_enc_reset (GstSrtEnc * srtenc);
 static void gst_srt_enc_set_property (GObject * object, guint prop_id,
     const GValue * value, GParamSpec * pspec);
 
-GST_BOILERPLATE (GstSrtEnc, gst_srt_enc, GstElement, GST_TYPE_ELEMENT);
+#define parent_class gst_srt_enc_parent_class
+G_DEFINE_TYPE (GstSrtEnc, gst_srt_enc, GST_TYPE_ELEMENT);
 
-static gchar *
+static void
-gst_srt_enc_timestamp_to_string (GstClockTime timestamp)
+gst_srt_enc_append_timestamp_to_string (GstClockTime timestamp, GString * str)
 {
   guint h, m, s, ms;
 
@@ -72,65 +74,84 @@ gst_srt_enc_timestamp_to_string (GstClockTime timestamp)
   timestamp -= s * GST_SECOND;
   ms = timestamp / GST_MSECOND;
 
-  return g_strdup_printf ("%.2d:%.2d:%.2d,%.3d", h, m, s, ms);
+  g_string_append_printf (str, "%.2d:%.2d:%.2d,%.3d", h, m, s, ms);
-}
-
-static gchar *
-gst_srt_enc_timeconvertion (GstSrtEnc * srtenc, GstBuffer * buf)
-{
-  gchar *start_time =
-      gst_srt_enc_timestamp_to_string (GST_BUFFER_TIMESTAMP (buf) +
-      srtenc->timestamp);
-  gchar *stop_time =
-      gst_srt_enc_timestamp_to_string (GST_BUFFER_TIMESTAMP (buf) +
-      srtenc->timestamp + GST_BUFFER_DURATION (buf) + srtenc->duration);
-  gchar *string = g_strdup_printf ("%s --> %s\n", start_time, stop_time);
-
-  g_free (start_time);
-  g_free (stop_time);
-  return string;
 }
 
 static GstFlowReturn
-gst_srt_enc_chain (GstPad * pad, GstBuffer * buf)
+gst_srt_enc_chain (GstPad * pad, GstObject * parent, GstBuffer * buf)
 {
-  GstSrtEnc *srtenc;
+  GstSrtEnc *srtenc = GST_SRT_ENC (parent);
+  GstClockTime ts, dur = GST_SECOND;
   GstBuffer *new_buffer;
-  gchar *timing;
+  GstMapInfo map_info;
-  gchar *string;
+  GString *s;
+  gsize buf_size;
 
-  srtenc = GST_SRT_ENC (gst_pad_get_parent_element (pad));
+  gst_object_sync_values (GST_OBJECT (srtenc), GST_BUFFER_PTS (buf));
-  gst_object_sync_values (GST_OBJECT (srtenc), GST_BUFFER_TIMESTAMP (buf));
+
-  timing = gst_srt_enc_timeconvertion (srtenc, buf);
+  ts = GST_BUFFER_PTS (buf) + srtenc->timestamp;
-  string = g_strdup_printf ("%d\n%s", srtenc->counter++, timing);
+  if (GST_BUFFER_DURATION_IS_VALID (buf))
-  g_free (timing);
+    dur = GST_BUFFER_DURATION (buf) + srtenc->duration;
-  new_buffer =
+  else if (srtenc->duration > 0)
-      gst_buffer_new_and_alloc (strlen (string) + GST_BUFFER_SIZE (buf) + 2);
+    dur = srtenc->duration;
-  memcpy (GST_BUFFER_DATA (new_buffer), string, strlen (string));
+  else
-  memcpy (GST_BUFFER_DATA (new_buffer) + strlen (string), GST_BUFFER_DATA (buf),
+    dur = GST_SECOND;
-      GST_BUFFER_SIZE (buf));
+
-  memcpy (GST_BUFFER_DATA (new_buffer) + GST_BUFFER_SIZE (new_buffer) - 2,
+  buf_size = gst_buffer_get_size (buf);
-      "\n\n", 2);
+  s = g_string_sized_new (10 + 50 + buf_size + 2 + 1);
-  g_free (string);
+
+  /* stanza count */
+  g_string_append_printf (s, "%d\n", srtenc->counter++);
+
+  /* start_time --> end_time */
+  gst_srt_enc_append_timestamp_to_string (ts, s);
+  g_string_append_printf (s, " --> ");
+  gst_srt_enc_append_timestamp_to_string (ts + dur, s);
+  g_string_append_c (s, '\n');
+
+  /* text */
+  if (gst_buffer_map (buf, &map_info, GST_MAP_READ)) {
+    g_string_append_len (s, (const gchar *) map_info.data, map_info.size);
+    gst_buffer_unmap (buf, &map_info);
+  }
+
+  g_string_append (s, "\n\n");
+
+  buf_size = s->len;
+  new_buffer = gst_buffer_new_wrapped (g_string_free (s, FALSE), buf_size);
+
+  GST_BUFFER_TIMESTAMP (new_buffer) = GST_BUFFER_TIMESTAMP (buf);
+  GST_BUFFER_DURATION (new_buffer) = GST_BUFFER_DURATION (buf);
 
   gst_buffer_unref (buf);
 
   return gst_pad_push (srtenc->srcpad, new_buffer);
 }
 
-static void
+static gboolean
-gst_srt_enc_base_init (gpointer klass)
+gst_srt_enc_event (GstPad * pad, GstObject * parent, GstEvent * event)
 {
-  GstElementClass *element_class = GST_ELEMENT_CLASS (klass);
+  GstSrtEnc *srtenc = GST_SRT_ENC (parent);
+  gboolean ret;
 
-  gst_element_class_add_pad_template (element_class,
+  switch (GST_EVENT_TYPE (event)) {
-      gst_static_pad_template_get (&sink_template));
+    case GST_EVENT_CAPS:
-  gst_element_class_add_pad_template (element_class,
+    {
-      gst_static_pad_template_get (&src_template));
+      GstCaps *caps;
 
-  gst_element_class_set_static_metadata (element_class,
+      caps = gst_static_pad_template_get_caps (&src_template);
-      "Srt encoder", "Codec/Encoder/Subtitle",
+      gst_pad_set_caps (srtenc->srcpad, caps);
-      "Srt subtitle encoder", "Thijs Vermeir <thijsvermeir@gmail.com>");
+      gst_caps_unref (caps);
+      gst_event_unref (event);
+      ret = TRUE;
+      break;
+    }
+    default:
+      ret = gst_pad_event_default (pad, parent, event);
+      break;
+  }
+
+  return ret;
 }
 
 static void
@@ -225,18 +246,31 @@ gst_srt_enc_class_init (GstSrtEncClass * klass)
           0,
           G_PARAM_READWRITE | GST_PARAM_CONTROLLABLE | G_PARAM_STATIC_STRINGS));
 
+  gst_element_class_add_pad_template (element_class,
+      gst_static_pad_template_get (&sink_template));
+  gst_element_class_add_pad_template (element_class,
+      gst_static_pad_template_get (&src_template));
+
+  gst_element_class_set_static_metadata (element_class,
+      "Srt encoder", "Codec/Encoder/Subtitle",
+      "Srt subtitle encoder", "Thijs Vermeir <thijsvermeir@gmail.com>");
+
   GST_DEBUG_CATEGORY_INIT (srtenc_debug, "srtenc", 0,
       "SubRip subtitle encoder");
 }
 
 static void
-gst_srt_enc_init (GstSrtEnc * srtenc, GstSrtEncClass * klass)
+gst_srt_enc_init (GstSrtEnc * srtenc)
 {
   gst_srt_enc_reset (srtenc);
 
   srtenc->srcpad = gst_pad_new_from_static_template (&src_template, "src");
   gst_element_add_pad (GST_ELEMENT (srtenc), srtenc->srcpad);
+
   srtenc->sinkpad = gst_pad_new_from_static_template (&sink_template, "sink");
+  gst_pad_set_chain_function (srtenc->sinkpad,
+      GST_DEBUG_FUNCPTR (gst_srt_enc_chain));
+  gst_pad_set_event_function (srtenc->sinkpad,
+      GST_DEBUG_FUNCPTR (gst_srt_enc_event));
   gst_element_add_pad (GST_ELEMENT (srtenc), srtenc->sinkpad);
-  gst_pad_set_chain_function (srtenc->sinkpad, gst_srt_enc_chain);
 }

gst/subenc/gstwebvttenc.c

@@ -21,10 +21,10 @@
 #ifdef HAVE_CONFIG_H
 #include "config.h"
 #endif
-#include "string.h"
+
+#include <string.h>
 
 #include "gstwebvttenc.h"
-#include <gst/controller/gstcontroller.h>
 
 GST_DEBUG_CATEGORY_STATIC (webvttenc_debug);
 #define GST_CAT_DEFAULT webvttenc_debug
@@ -39,27 +39,29 @@ enum
 static GstStaticPadTemplate src_template = GST_STATIC_PAD_TEMPLATE ("src",
     GST_PAD_SRC,
     GST_PAD_ALWAYS,
-    GST_STATIC_CAPS ("text/webvtt"));
+    GST_STATIC_CAPS ("text/vtt"));
 
 static GstStaticPadTemplate sink_template = GST_STATIC_PAD_TEMPLATE ("sink",
     GST_PAD_SINK,
     GST_PAD_ALWAYS,
-    GST_STATIC_CAPS ("text/plain; text/x-pango-markup"));
+    GST_STATIC_CAPS ("text/x-raw, format = { pango-markup, utf8 }"));
 
-static GstFlowReturn gst_webvtt_enc_chain (GstPad * pad, GstBuffer * buf);
+static GstFlowReturn gst_webvtt_enc_chain (GstPad * pad, GstObject * parent,
-static gchar *gst_webvtt_enc_timeconvertion (GstWebvttEnc * webvttenc,
     GstBuffer * buf);
-static gchar *gst_webvtt_enc_timestamp_to_string (GstClockTime timestamp);
+static void gst_webvtt_enc_append_timestamp_to_string (GstClockTime timestamp,
+    GString * str);
 static void gst_webvtt_enc_get_property (GObject * object, guint prop_id,
     GValue * value, GParamSpec * pspec);
 static void gst_webvtt_enc_reset (GstWebvttEnc * webvttenc);
 static void gst_webvtt_enc_set_property (GObject * object, guint prop_id,
     const GValue * value, GParamSpec * pspec);
 
-GST_BOILERPLATE (GstWebvttEnc, gst_webvtt_enc, GstElement, GST_TYPE_ELEMENT);
+#define parent_class gst_webvtt_enc_parent_class
+G_DEFINE_TYPE (GstWebvttEnc, gst_webvtt_enc, GST_TYPE_ELEMENT);
 
-static gchar *
+static void
-gst_webvtt_enc_timestamp_to_string (GstClockTime timestamp)
+gst_webvtt_enc_append_timestamp_to_string (GstClockTime timestamp,
+    GString * str)
 {
   guint h, m, s, ms;
 
@@ -74,97 +76,102 @@ gst_webvtt_enc_timestamp_to_string (GstClockTime timestamp)
   timestamp -= s * GST_SECOND;
   ms = timestamp / GST_MSECOND;
 
-  return g_strdup_printf ("%02d:%02d:%02d.%03d", h, m, s, ms);
+  g_string_append_printf (str, "%02d:%02d:%02d.%03d", h, m, s, ms);
-}
-
-static gchar *
-gst_webvtt_enc_timeconvertion (GstWebvttEnc * webvttenc, GstBuffer * buf)
-{
-  gchar *start_time;
-  gchar *stop_time;
-  gchar *string;
-
-  start_time = gst_webvtt_enc_timestamp_to_string (GST_BUFFER_TIMESTAMP (buf) +
-      webvttenc->timestamp);
-  if (GST_CLOCK_TIME_IS_VALID (GST_BUFFER_DURATION (buf))) {
-    stop_time = gst_webvtt_enc_timestamp_to_string (GST_BUFFER_TIMESTAMP (buf) +
-        webvttenc->timestamp + GST_BUFFER_DURATION (buf) + webvttenc->duration);
-  } else {
-    stop_time = gst_webvtt_enc_timestamp_to_string (GST_BUFFER_TIMESTAMP (buf) +
-        webvttenc->timestamp + webvttenc->duration);
-  }
-  string = g_strdup_printf ("%s --> %s\n", start_time, stop_time);
-
-  g_free (start_time);
-  g_free (stop_time);
-  return string;
 }
 
 static GstFlowReturn
-gst_webvtt_enc_chain (GstPad * pad, GstBuffer * buf)
+gst_webvtt_enc_chain (GstPad * pad, GstObject * parent, GstBuffer * buf)
 {
-  GstWebvttEnc *webvttenc;
+  GstWebvttEnc *webvttenc = GST_WEBVTT_ENC (parent);
+  GstClockTime ts, dur = GST_SECOND;
   GstBuffer *new_buffer;
-  gchar *timing;
+  GstMapInfo map_info;
   GstFlowReturn ret;
-
+  GString *s;
-  webvttenc = GST_WEBVTT_ENC (gst_pad_get_parent_element (pad));
+  gsize buf_size;
 
   if (!webvttenc->pushed_header) {
     const char *header = "WEBVTT\n\n";
 
-    new_buffer = gst_buffer_new_and_alloc (strlen (header));
+    new_buffer = gst_buffer_new_wrapped (g_strdup (header), strlen (header));
-    memcpy (GST_BUFFER_DATA (new_buffer), header, strlen (header));
 
-    GST_BUFFER_TIMESTAMP (new_buffer) = GST_CLOCK_TIME_NONE;
+    GST_BUFFER_PTS (new_buffer) = GST_CLOCK_TIME_NONE;
     GST_BUFFER_DURATION (new_buffer) = GST_CLOCK_TIME_NONE;
 
     ret = gst_pad_push (webvttenc->srcpad, new_buffer);
-    if (ret != GST_FLOW_OK) {
+
+    if (ret != GST_FLOW_OK)
       goto out;
-    }
 
     webvttenc->pushed_header = TRUE;
   }
 
-  gst_object_sync_values (GST_OBJECT (webvttenc), GST_BUFFER_TIMESTAMP (buf));
+  gst_object_sync_values (GST_OBJECT (webvttenc), GST_BUFFER_PTS (buf));
 
-  timing = gst_webvtt_enc_timeconvertion (webvttenc, buf);
+  ts = GST_BUFFER_PTS (buf) + webvttenc->timestamp;
-  new_buffer =
+  if (GST_BUFFER_DURATION_IS_VALID (buf))
-      gst_buffer_new_and_alloc (strlen (timing) + GST_BUFFER_SIZE (buf) + 1);
+    dur = GST_BUFFER_DURATION (buf) + webvttenc->duration;
-  memcpy (GST_BUFFER_DATA (new_buffer), timing, strlen (timing));
+  else if (webvttenc->duration > 0)
-  memcpy (GST_BUFFER_DATA (new_buffer) + strlen (timing), GST_BUFFER_DATA (buf),
+    dur = webvttenc->duration;
-      GST_BUFFER_SIZE (buf));
+  else
-  memcpy (GST_BUFFER_DATA (new_buffer) + GST_BUFFER_SIZE (new_buffer) - 1,
+    dur = GST_SECOND;
-      "\n", 1);
+
-  g_free (timing);
+  buf_size = gst_buffer_get_size (buf);
+  s = g_string_sized_new (50 + buf_size + 1 + 1);
+
+  /* start_time --> end_time */
+  gst_webvtt_enc_append_timestamp_to_string (ts, s);
+  g_string_append_printf (s, " --> ");
+  gst_webvtt_enc_append_timestamp_to_string (ts + dur, s);
+  g_string_append_c (s, '\n');
+
+  /* text */
+  if (gst_buffer_map (buf, &map_info, GST_MAP_READ)) {
+    g_string_append_len (s, (const gchar *) map_info.data, map_info.size);
+    gst_buffer_unmap (buf, &map_info);
+  }
+
+  g_string_append_c (s, '\n');
+
+  buf_size = s->len;
+  new_buffer = gst_buffer_new_wrapped (g_string_free (s, FALSE), buf_size);
 
   GST_BUFFER_TIMESTAMP (new_buffer) = GST_BUFFER_TIMESTAMP (buf);
   GST_BUFFER_DURATION (new_buffer) = GST_BUFFER_DURATION (buf);
 
-
   ret = gst_pad_push (webvttenc->srcpad, new_buffer);
 
 out:
+
   gst_buffer_unref (buf);
-  gst_object_unref (webvttenc);
 
   return ret;
 }
 
-static void
+static gboolean
-gst_webvtt_enc_base_init (gpointer klass)
+gst_webvtt_enc_event (GstPad * pad, GstObject * parent, GstEvent * event)
 {
-  GstElementClass *element_class = GST_ELEMENT_CLASS (klass);
+  GstWebvttEnc *webvttenc = GST_WEBVTT_ENC (parent);
+  gboolean ret;
 
-  gst_element_class_add_pad_template (element_class,
+  switch (GST_EVENT_TYPE (event)) {
-      gst_static_pad_template_get (&sink_template));
+    case GST_EVENT_CAPS:
-  gst_element_class_add_pad_template (element_class,
+    {
-      gst_static_pad_template_get (&src_template));
+      GstCaps *caps;
 
-  gst_element_class_set_static_metadata (element_class,
+      caps = gst_static_pad_template_get_caps (&src_template);
-      "WebVTT encoder", "Codec/Encoder/Subtitle",
+      gst_pad_set_caps (webvttenc->srcpad, caps);
-      "WebVTT subtitle encoder", "David Schleef <ds@schleef.org>");
+      gst_caps_unref (caps);
+      gst_event_unref (event);
+      ret = TRUE;
+      break;
+    }
+    default:
+      ret = gst_pad_event_default (pad, parent, event);
+      break;
+  }
+
+  return ret;
 }
 
 static void
@@ -259,12 +266,21 @@ gst_webvtt_enc_class_init (GstWebvttEncClass * klass)
           0,
           G_PARAM_READWRITE | GST_PARAM_CONTROLLABLE | G_PARAM_STATIC_STRINGS));
 
+  gst_element_class_add_pad_template (element_class,
+      gst_static_pad_template_get (&sink_template));
+  gst_element_class_add_pad_template (element_class,
+      gst_static_pad_template_get (&src_template));
+
+  gst_element_class_set_static_metadata (element_class,
+      "WebVTT encoder", "Codec/Encoder/Subtitle",
+      "WebVTT subtitle encoder", "David Schleef <ds@schleef.org>");
+
   GST_DEBUG_CATEGORY_INIT (webvttenc_debug, "webvttenc", 0,
       "SubRip subtitle encoder");
 }
 
 static void
-gst_webvtt_enc_init (GstWebvttEnc * webvttenc, GstWebvttEncClass * klass)
+gst_webvtt_enc_init (GstWebvttEnc * webvttenc)
 {
   gst_webvtt_enc_reset (webvttenc);
 
@@ -272,6 +288,9 @@ gst_webvtt_enc_init (GstWebvttEnc * webvttenc, GstWebvttEncClass * klass)
   gst_element_add_pad (GST_ELEMENT (webvttenc), webvttenc->srcpad);
   webvttenc->sinkpad =
       gst_pad_new_from_static_template (&sink_template, "sink");
+  gst_pad_set_chain_function (webvttenc->sinkpad,
+      GST_DEBUG_FUNCPTR (gst_webvtt_enc_chain));
+  gst_pad_set_event_function (webvttenc->sinkpad,
+      GST_DEBUG_FUNCPTR (gst_webvtt_enc_event));
   gst_element_add_pad (GST_ELEMENT (webvttenc), webvttenc->sinkpad);
-  gst_pad_set_chain_function (webvttenc->sinkpad, gst_webvtt_enc_chain);
 }