gst-indent
[platform/upstream/gst-plugins-good.git] / ext / libcaca / gstcacasink.c
index 2b83084..4cb64e4 100644 (file)
@@ -23,6 +23,7 @@
 
 #include <string.h>
 #include <sys/time.h>
+#include <gst/navigation/navigation.h>
 
 #include "gstcacasink.h"
 
@@ -35,49 +36,49 @@ static GstElementDetails gst_cacasink_details = {
 };
 
 /* cacasink signals and args */
-enum {
-  SIGNAL_FRAME_DISPLAYED,
-  SIGNAL_HAVE_SIZE,
+enum
+{
   LAST_SIGNAL
 };
 
 
-enum {
+enum
+{
   ARG_0,
-  ARG_WIDTH,
-  ARG_HEIGHT,
+  ARG_SCREEN_WIDTH,
+  ARG_SCREEN_HEIGHT,
   ARG_DITHER,
-  ARG_FRAMES_DISPLAYED,
-  ARG_FRAME_TIME,
+  ARG_ANTIALIASING
 };
 
-GST_PAD_TEMPLATE_FACTORY (sink_template,
-  "sink",
-  GST_PAD_SINK,
-  GST_PAD_ALWAYS,
-  gst_caps_new (
-    "cacasink_caps",
-    "video/x-raw-rgb",
-    GST_VIDEO_RGB_PAD_TEMPLATE_PROPS_24_32
-  )
-)
-
-static void    gst_cacasink_base_init  (gpointer g_class);
-static void    gst_cacasink_class_init (GstCACASinkClass *klass);
-static void    gst_cacasink_init               (GstCACASink *cacasink);
-
-static void    gst_cacasink_set_clock  (GstElement *element, GstClock *clock);
-static void    gst_cacasink_chain      (GstPad *pad, GstData *_data);
-
-static void    gst_cacasink_set_property       (GObject *object, guint prop_id, 
-                                        const GValue *value, GParamSpec *pspec);
-static void    gst_cacasink_get_property       (GObject *object, guint prop_id, 
-                                        GValue *value, GParamSpec *pspec);
-
-static GstElementStateReturn gst_cacasink_change_state (GstElement *element);
+static GstStaticPadTemplate sink_template = GST_STATIC_PAD_TEMPLATE ("sink",
+    GST_PAD_SINK,
+    GST_PAD_ALWAYS,
+    GST_STATIC_CAPS (GST_VIDEO_CAPS_RGB ";" GST_VIDEO_CAPS_RGBx ";"
+       GST_VIDEO_CAPS_RGB_16 ";" GST_VIDEO_CAPS_RGB_15)
+    );
+
+static void gst_cacasink_base_init (gpointer g_class);
+static void gst_cacasink_class_init (GstCACASinkClass * klass);
+static void gst_cacasink_init (GstCACASink * cacasink);
+static void gst_cacasink_interface_init (GstImplementsInterfaceClass * klass);
+static gboolean gst_cacasink_interface_supported (GstImplementsInterface *
+    iface, GType type);
+static void gst_cacasink_navigation_init (GstNavigationInterface * iface);
+static void gst_cacasink_navigation_send_event (GstNavigation * navigation,
+    GstStructure * structure);
+
+static void gst_cacasink_chain (GstPad * pad, GstData * _data);
+
+static void gst_cacasink_set_property (GObject * object, guint prop_id,
+    const GValue * value, GParamSpec * pspec);
+static void gst_cacasink_get_property (GObject * object, guint prop_id,
+    GValue * value, GParamSpec * pspec);
+static void gst_cacasink_dispose (GObject * object);
+
+static GstElementStateReturn gst_cacasink_change_state (GstElement * element);
 
 static GstElementClass *parent_class = NULL;
-static guint gst_cacasink_signals[LAST_SIGNAL] = { 0 };
 
 GType
 gst_cacasink_get_type (void)
@@ -86,17 +87,37 @@ gst_cacasink_get_type (void)
 
   if (!cacasink_type) {
     static const GTypeInfo cacasink_info = {
-      sizeof(GstCACASinkClass),
+      sizeof (GstCACASinkClass),
       gst_cacasink_base_init,
       NULL,
       (GClassInitFunc) gst_cacasink_class_init,
       NULL,
       NULL,
-      sizeof(GstCACASink),
+      sizeof (GstCACASink),
       0,
-      (GInstanceInitFunc)gst_cacasink_init,
+      (GInstanceInitFunc) gst_cacasink_init,
+    };
+
+    static const GInterfaceInfo iface_info = {
+      (GInterfaceInitFunc) gst_cacasink_interface_init,
+      NULL,
+      NULL,
+    };
+
+    static const GInterfaceInfo navigation_info = {
+      (GInterfaceInitFunc) gst_cacasink_navigation_init,
+      NULL,
+      NULL,
     };
-    cacasink_type = g_type_register_static(GST_TYPE_ELEMENT, "GstCACASink", &cacasink_info, 0);
+
+    cacasink_type =
+       g_type_register_static (GST_TYPE_VIDEOSINK, "GstCACASink",
+       &cacasink_info, 0);
+
+    g_type_add_interface_static (cacasink_type, GST_TYPE_IMPLEMENTS_INTERFACE,
+       &iface_info);
+    g_type_add_interface_static (cacasink_type, GST_TYPE_NAVIGATION,
+       &navigation_info);
   }
   return cacasink_type;
 }
@@ -106,18 +127,20 @@ static GType
 gst_cacasink_dither_get_type (void)
 {
   static GType dither_type = 0;
+
   if (!dither_type) {
     GEnumValue *dithers;
     gint n_dithers;
     gint i;
-    gchar caca_dithernames[] = {
-       "NONE", "ORDERED2", "ORDERED4", "ORDERED8", "RANDOM", NULL};
+    gchar *caca_dithernames[] = {
+      "NONE", "ORDERED2", "ORDERED4", "ORDERED8", "RANDOM", NULL
+    };
 
     n_dithers = 5;
-    
-    dithers = g_new0(GEnumValue, n_dithers + 1);
 
-    for (i = 0; i < n_dithers; i++){
+    dithers = g_new0 (GEnumValue, n_dithers + 1);
+
+    for (i = 0; i < n_dithers; i++) {
       dithers[i].value = i;
       dithers[i].value_name = g_strdup (caca_dithernames[i]);
       dithers[i].value_nick = g_strdup (caca_dithernames[i]);
@@ -136,121 +159,171 @@ gst_cacasink_base_init (gpointer g_class)
 {
   GstElementClass *element_class = GST_ELEMENT_CLASS (g_class);
 
-  gst_element_class_add_pad_template (element_class,
-               GST_PAD_TEMPLATE_GET (sink_template));
   gst_element_class_set_details (element_class, &gst_cacasink_details);
+  gst_element_class_add_pad_template (element_class,
+      gst_static_pad_template_get (&sink_template));
 }
 
 static void
-gst_cacasink_class_init (GstCACASinkClass *klass)
+gst_cacasink_class_init (GstCACASinkClass * klass)
 {
   GObjectClass *gobject_class;
   GstElementClass *gstelement_class;
+  GstVideoSinkClass *gstvs_class;
 
-  gobject_class = (GObjectClass*)klass;
-  gstelement_class = (GstElementClass*)klass;
+  gobject_class = (GObjectClass *) klass;
+  gstelement_class = (GstElementClass *) klass;
+  gstvs_class = (GstVideoSinkClass *) klass;
 
   parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
 
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_WIDTH,
-    g_param_spec_int("width","width","width",
-                     G_MININT,G_MAXINT,0,G_PARAM_READABLE)); /* CHECKME */
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_HEIGHT,
-    g_param_spec_int("height","height","height",
-                     G_MININT,G_MAXINT,0,G_PARAM_READABLE)); /* CHECKME */
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_DITHER,
-    g_param_spec_enum("dither","dither","dither",
-                      GST_TYPE_CACADITHER,0,G_PARAM_READWRITE)); /* CHECKME! */
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_FRAMES_DISPLAYED,
-    g_param_spec_int("frames_displayed","frames_displayed","frames_displayed",
-                     G_MININT,G_MAXINT,0,G_PARAM_READABLE)); /* CHECKME */
-  g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_FRAME_TIME,
-    g_param_spec_int("frame_time","frame_time","frame_time",
-                     G_MININT,G_MAXINT,0,G_PARAM_READABLE)); /* CHECKME */
+  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_SCREEN_WIDTH, g_param_spec_int ("screen_width", "screen_width", "screen_width", G_MININT, G_MAXINT, 0, G_PARAM_READABLE));      /* CHECKME */
+  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_SCREEN_HEIGHT, g_param_spec_int ("screen_height", "screen_height", "screen_height", G_MININT, G_MAXINT, 0, G_PARAM_READABLE));  /* CHECKME */
+  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_DITHER,
+      g_param_spec_enum ("dither", "Dither Type", "Set type of Dither",
+         GST_TYPE_CACADITHER, 0, G_PARAM_READWRITE));
+  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_ANTIALIASING,
+      g_param_spec_boolean ("anti_aliasing", "Anti-Aliasing",
+         "Enables Anti-Aliasing", TRUE, G_PARAM_READWRITE));
 
   gobject_class->set_property = gst_cacasink_set_property;
   gobject_class->get_property = gst_cacasink_get_property;
-
-  gst_cacasink_signals[SIGNAL_FRAME_DISPLAYED] =
-    g_signal_new ("frame_displayed", G_TYPE_FROM_CLASS(klass), G_SIGNAL_RUN_LAST,
-                   G_STRUCT_OFFSET (GstCACASinkClass, frame_displayed), NULL, NULL,
-                   g_cclosure_marshal_VOID__VOID, G_TYPE_NONE, 0);
-  gst_cacasink_signals[SIGNAL_HAVE_SIZE] =
-    g_signal_new ("have_size", G_TYPE_FROM_CLASS(klass), G_SIGNAL_RUN_LAST,
-                   G_STRUCT_OFFSET (GstCACASinkClass, have_size), NULL, NULL,
-                   gst_marshal_VOID__INT_INT, G_TYPE_NONE, 2,
-                   G_TYPE_UINT, G_TYPE_UINT);
+  gobject_class->dispose = gst_cacasink_dispose;
 
   gstelement_class->change_state = gst_cacasink_change_state;
-  gstelement_class->set_clock    = gst_cacasink_set_clock;
+}
+
+static void
+gst_cacasink_interface_init (GstImplementsInterfaceClass * klass)
+{
+  klass->supported = gst_cacasink_interface_supported;
+}
+
+static gboolean
+gst_cacasink_interface_supported (GstImplementsInterface * iface, GType type)
+{
+  g_assert (type == GST_TYPE_NAVIGATION);
+
+  return (GST_STATE (iface) != GST_STATE_NULL);
+}
+
+static void
+gst_cacasink_navigation_init (GstNavigationInterface * iface)
+{
+  iface->send_event = gst_cacasink_navigation_send_event;
+}
+
+static void
+gst_cacasink_navigation_send_event (GstNavigation * navigation,
+    GstStructure * structure)
+{
+  GstCACASink *cacasink = GST_CACASINK (navigation);
+  GstEvent *event;
+
+  event = gst_event_new (GST_EVENT_NAVIGATION);
+  /*GST_EVENT_TIMESTAMP (event) = 0; */
+  event->event_data.structure.structure = structure;
+
+  /* FIXME 
+   * Obviously, the pointer x,y coordinates need to be adjusted by the
+   * window size and relation to the bounding window. */
+
+  gst_pad_send_event (gst_pad_get_peer (GST_VIDEOSINK_PAD (cacasink)), event);
 }
 
 static GstPadLinkReturn
-gst_cacasink_sinkconnect (GstPad *pad, GstCaps *caps)
+gst_cacasink_sinkconnect (GstPad * pad, const GstCaps * caps)
 {
   GstCACASink *cacasink;
+  GstStructure *structure;
 
   cacasink = GST_CACASINK (gst_pad_get_parent (pad));
 
-  if (!GST_CAPS_IS_FIXED (caps))
-    return GST_PAD_LINK_DELAYED;
-  
-  gst_caps_get_int (caps, "width", &cacasink->image_width);
-  gst_caps_get_int (caps, "height", &cacasink->image_height);
-  gst_caps_get_int (caps, "bpp", &cacasink->bpp);
-  gst_caps_get_int (caps, "red_mask", &cacasink->red_mask);
-  gst_caps_get_int (caps, "green_mask", &cacasink->green_mask);
-  gst_caps_get_int (caps, "blue_mask", &cacasink->blue_mask);
-
-  g_signal_emit( G_OBJECT (cacasink), gst_cacasink_signals[SIGNAL_HAVE_SIZE], 0,
-                cacasink->image_width, cacasink->image_height);
-
-  /*if (cacasink->bitmap != NULL) {
-    caca_free_bitmap (cacasink->bitmap);
+  /*if (!GST_CAPS_IS_FIXED (caps))
+     return GST_PAD_LINK_DELAYED; */
+
+  structure = gst_caps_get_structure (caps, 0);
+  gst_structure_get_int (structure, "width", &(GST_VIDEOSINK_WIDTH (cacasink)));
+  gst_structure_get_int (structure, "height",
+      &(GST_VIDEOSINK_HEIGHT (cacasink)));
+  gst_structure_get_int (structure, "bpp", &cacasink->bpp);
+  gst_structure_get_int (structure, "red_mask", &cacasink->red_mask);
+  gst_structure_get_int (structure, "green_mask", &cacasink->green_mask);
+  gst_structure_get_int (structure, "blue_mask", &cacasink->blue_mask);
+
+  if (cacasink->bpp == 24) {
+    cacasink->red_mask = GUINT32_FROM_BE (cacasink->red_mask) >> 8;
+    cacasink->green_mask = GUINT32_FROM_BE (cacasink->green_mask) >> 8;
+    cacasink->blue_mask = GUINT32_FROM_BE (cacasink->blue_mask) >> 8;
   }
 
-  caca->bitmap = caca_create_bitmap (cacasink->bpp, cacasink->image_width, cacasink->image_height, cacasink->image_width * cacasink->bpp/8, cacasink->red_mask, cacasink->green_mask, cacasink->blue_mask);*/
+  else if (cacasink->bpp == 32) {
+    cacasink->red_mask = GUINT32_FROM_BE (cacasink->red_mask);
+    cacasink->green_mask = GUINT32_FROM_BE (cacasink->green_mask);
+    cacasink->blue_mask = GUINT32_FROM_BE (cacasink->blue_mask);
+  }
 
-  return GST_PAD_LINK_OK;
-}
+  else if (cacasink->bpp == 16 || cacasink->bpp == 15) {
+    cacasink->red_mask = GUINT16_FROM_BE (cacasink->red_mask);
+    cacasink->green_mask = GUINT16_FROM_BE (cacasink->green_mask);
+    cacasink->blue_mask = GUINT16_FROM_BE (cacasink->blue_mask);
+  }
 
-static void
-gst_cacasink_set_clock (GstElement *element, GstClock *clock)
-{
-  GstCACASink *cacasink = GST_CACASINK (element);
+  if (cacasink->bitmap) {
+    caca_free_bitmap (cacasink->bitmap);
+  }
+
+  cacasink->bitmap = caca_create_bitmap (cacasink->bpp,
+      GST_VIDEOSINK_WIDTH (cacasink),
+      GST_VIDEOSINK_HEIGHT (cacasink),
+      GST_VIDEOSINK_WIDTH (cacasink) * cacasink->bpp / 8,
+      cacasink->red_mask, cacasink->green_mask, cacasink->blue_mask, 0);
+
+  if (!cacasink->bitmap) {
+    return GST_PAD_LINK_DELAYED;
+  }
 
-  cacasink->clock = clock;
+  return GST_PAD_LINK_OK;
 }
 
 static void
-gst_cacasink_init (GstCACASink *cacasink)
+gst_cacasink_init (GstCACASink * cacasink)
 {
-  cacasink->sinkpad = gst_pad_new_from_template (
-                 GST_PAD_TEMPLATE_GET (sink_template), "sink");
-  gst_element_add_pad (GST_ELEMENT (cacasink), cacasink->sinkpad);
-  gst_pad_set_chain_function (cacasink->sinkpad, gst_cacasink_chain);
-  gst_pad_set_link_function (cacasink->sinkpad, gst_cacasink_sinkconnect);
-
-  cacasink->screen_width = -1;
-  cacasink->screen_height = -1;
-  cacasink->image_width = GST_CACA_DEFAULT_IMAGE_WIDTH;
-  cacasink->image_height = GST_CACA_DEFAULT_IMAGE_HEIGHT;
+  GST_VIDEOSINK_PAD (cacasink) =
+      gst_pad_new_from_template (gst_static_pad_template_get (&sink_template),
+      "sink");
+  gst_element_add_pad (GST_ELEMENT (cacasink), GST_VIDEOSINK_PAD (cacasink));
+  gst_pad_set_chain_function (GST_VIDEOSINK_PAD (cacasink), gst_cacasink_chain);
+  gst_pad_set_link_function (GST_VIDEOSINK_PAD (cacasink),
+      gst_cacasink_sinkconnect);
+
+  cacasink->screen_width = GST_CACA_DEFAULT_SCREEN_WIDTH;
+  cacasink->screen_height = GST_CACA_DEFAULT_SCREEN_HEIGHT;
   cacasink->bpp = GST_CACA_DEFAULT_BPP;
   cacasink->red_mask = GST_CACA_DEFAULT_RED_MASK;
   cacasink->green_mask = GST_CACA_DEFAULT_GREEN_MASK;
   cacasink->blue_mask = GST_CACA_DEFAULT_BLUE_MASK;
 
-  cacasink->clock = NULL;
   cacasink->bitmap = NULL;
+  caca_init ();
+
+  cacasink->screen_width = caca_get_width ();
+  cacasink->screen_height = caca_get_height ();
+  cacasink->antialiasing = TRUE;
+  caca_set_feature (CACA_ANTIALIASING_MAX);
+  cacasink->dither = 0;
+  caca_set_dithering (CACA_DITHERING_NONE);
 
-  GST_FLAG_SET(cacasink, GST_ELEMENT_THREAD_SUGGESTED);
+  GST_FLAG_SET (cacasink, GST_ELEMENT_THREAD_SUGGESTED);
 }
 
 static void
-gst_cacasink_chain (GstPad *pad, GstData *_data)
+gst_cacasink_chain (GstPad * pad, GstData * _data)
 {
   GstBuffer *buf = GST_BUFFER (_data);
   GstCACASink *cacasink;
+  GstClockTime time = GST_BUFFER_TIMESTAMP (buf);
+  gint64 jitter;
 
   g_return_if_fail (pad != NULL);
   g_return_if_fail (GST_IS_PAD (pad));
@@ -258,25 +331,46 @@ gst_cacasink_chain (GstPad *pad, GstData *_data)
 
   cacasink = GST_CACASINK (gst_pad_get_parent (pad));
 
-  GST_DEBUG ("videosink: clock wait: %" G_GUINT64_FORMAT, GST_BUFFER_TIMESTAMP(buf));
-
-  if (cacasink->clock) {
-    GstClockID id = gst_clock_new_single_shot_id (cacasink->clock, GST_BUFFER_TIMESTAMP(buf));
-    gst_element_clock_wait (GST_ELEMENT (cacasink), id, NULL);
-    gst_clock_id_free (id);
+  if (cacasink->id && GST_CLOCK_TIME_IS_VALID (time)) {
+    GST_DEBUG ("videosink: clock %s wait: %" G_GUINT64_FORMAT " %u",
+       GST_OBJECT_NAME (GST_VIDEOSINK_CLOCK (cacasink)),
+       time, GST_BUFFER_SIZE (buf));
+    gst_element_wait (GST_ELEMENT (cacasink), GST_BUFFER_TIMESTAMP (buf));
   }
 
-  caca_draw_bitmap (0, 0, cacasink->screen_width-1, cacasink->screen_height-1, cacasink->bitmap, GST_BUFFER_DATA (buf));
+  caca_clear ();
+  caca_draw_bitmap (0, 0, cacasink->screen_width - 1,
+      cacasink->screen_height - 1, cacasink->bitmap, GST_BUFFER_DATA (buf));
   caca_refresh ();
 
-  g_signal_emit(G_OBJECT(cacasink),gst_cacasink_signals[SIGNAL_FRAME_DISPLAYED], 0);
+  if (GST_VIDEOSINK_CLOCK (cacasink)) {
+    jitter = gst_clock_get_time (GST_VIDEOSINK_CLOCK (cacasink)) - time;
 
-  gst_buffer_unref(buf);
+    cacasink->correction = (cacasink->correction + jitter) >> 1;
+    cacasink->correction = 0;
+  }
+
+  gst_buffer_unref (buf);
 }
 
 
 static void
-gst_cacasink_set_property (GObject *object, guint prop_id, const GValue *value, GParamSpec *pspec)
+gst_cacasink_dispose (GObject * object)
+{
+  GstCACASink *cacasink = GST_CACASINK (object);
+
+  if (cacasink->bitmap) {
+    caca_free_bitmap (cacasink->bitmap);
+  }
+
+  caca_end ();
+
+  G_OBJECT_CLASS (parent_class)->dispose (object);
+}
+
+static void
+gst_cacasink_set_property (GObject * object, guint prop_id,
+    const GValue * value, GParamSpec * pspec)
 {
   GstCACASink *cacasink;
 
@@ -286,8 +380,20 @@ gst_cacasink_set_property (GObject *object, guint prop_id, const GValue *value,
   cacasink = GST_CACASINK (object);
 
   switch (prop_id) {
-    case ARG_DITHER: {
+    case ARG_DITHER:{
       cacasink->dither = g_value_get_enum (value);
+      caca_set_dithering (cacasink->dither + CACA_DITHERING_NONE);
+      break;
+    }
+    case ARG_ANTIALIASING:{
+      cacasink->antialiasing = g_value_get_boolean (value);
+      if (cacasink->antialiasing) {
+       caca_set_feature (CACA_ANTIALIASING_MAX);
+      }
+
+      else {
+       caca_set_feature (CACA_ANTIALIASING_MIN);
+      }
       break;
     }
     default:
@@ -296,35 +402,32 @@ gst_cacasink_set_property (GObject *object, guint prop_id, const GValue *value,
 }
 
 static void
-gst_cacasink_get_property (GObject *object, guint prop_id, GValue *value, GParamSpec *pspec)
+gst_cacasink_get_property (GObject * object, guint prop_id, GValue * value,
+    GParamSpec * pspec)
 {
   GstCACASink *cacasink;
 
   /* it's not null if we got it, but it might not be ours */
-  cacasink = GST_CACASINK(object);
+  cacasink = GST_CACASINK (object);
 
   switch (prop_id) {
-    case ARG_WIDTH: {
+    case ARG_SCREEN_WIDTH:{
       g_value_set_int (value, cacasink->screen_width);
       break;
     }
-    case ARG_HEIGHT: {
+    case ARG_SCREEN_HEIGHT:{
       g_value_set_int (value, cacasink->screen_height);
       break;
     }
-    case ARG_DITHER: {
+    case ARG_DITHER:{
       g_value_set_enum (value, cacasink->dither);
       break;
     }
-    case ARG_FRAMES_DISPLAYED: {
-      g_value_set_int (value, cacasink->frames_displayed);
+    case ARG_ANTIALIASING:{
+      g_value_set_boolean (value, cacasink->antialiasing);
       break;
     }
-    case ARG_FRAME_TIME: {
-      g_value_set_int (value, cacasink->frame_time/1000000);
-      break;
-    }
-    default: {
+    default:{
       G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
     }
@@ -332,24 +435,9 @@ gst_cacasink_get_property (GObject *object, guint prop_id, GValue *value, GParam
 }
 
 static gboolean
-gst_cacasink_open (GstCACASink *cacasink)
+gst_cacasink_open (GstCACASink * cacasink)
 {
-  g_return_val_if_fail (!GST_FLAG_IS_SET (cacasink ,GST_CACASINK_OPEN), FALSE);
-
-  caca_init ();
-
-  screen_width = caca_get_width ();
-  screen_height = caca_get_height ();
-  caca_set_dithering (cacasink->dither);
-
-  caca->bitmap = caca_create_bitmap (
-                       cacasink->bpp, 
-                       cacasink->image_width, 
-                       cacasink->image_height, 
-                       cacasink->image_width * cacasink->bpp/8, 
-                       cacasink->red_mask, 
-                       cacasink->green_mask, 
-                       cacasink->blue_mask);
+  g_return_val_if_fail (!GST_FLAG_IS_SET (cacasink, GST_CACASINK_OPEN), FALSE);
 
   GST_FLAG_SET (cacasink, GST_CACASINK_OPEN);
 
@@ -357,19 +445,15 @@ gst_cacasink_open (GstCACASink *cacasink)
 }
 
 static void
-gst_cacasink_close (GstCACASink *cacasink)
+gst_cacasink_close (GstCACASink * cacasink)
 {
-  g_return_if_fail (GST_FLAG_IS_SET (cacasink ,GST_CACASINK_OPEN));
-
-  caca_free_bitmap (cacasink->bitmap);
-  cacasink->bitmap = NULL;
-  caca_end ();
+  g_return_if_fail (GST_FLAG_IS_SET (cacasink, GST_CACASINK_OPEN));
 
   GST_FLAG_UNSET (cacasink, GST_CACASINK_OPEN);
 }
 
 static GstElementStateReturn
-gst_cacasink_change_state (GstElement *element)
+gst_cacasink_change_state (GstElement * element)
 {
   g_return_val_if_fail (GST_IS_CACASINK (element), GST_STATE_FAILURE);
 
@@ -379,7 +463,7 @@ gst_cacasink_change_state (GstElement *element)
   } else {
     if (!GST_FLAG_IS_SET (element, GST_CACASINK_OPEN)) {
       if (!gst_cacasink_open (GST_CACASINK (element)))
-        return GST_STATE_FAILURE;
+       return GST_STATE_FAILURE;
     }
   }
 
@@ -390,22 +474,21 @@ gst_cacasink_change_state (GstElement *element)
 }
 
 static gboolean
-plugin_init (GstPlugin *plugin)
+plugin_init (GstPlugin * plugin)
 {
-  if (!gst_element_register (plugin, "cacasink", GST_RANK_NONE, GST_TYPE_CACASINK))
+  /* Loading the library containing GstVideoSink, our parent object */
+  if (!gst_library_load ("gstvideo"))
+    return FALSE;
+
+  if (!gst_element_register (plugin, "cacasink", GST_RANK_NONE,
+         GST_TYPE_CACASINK))
     return FALSE;
 
   return TRUE;
 }
 
-GST_PLUGIN_DEFINE (
-  GST_VERSION_MAJOR,
-  GST_VERSION_MINOR,
-  "cacasink",
-  "Colored ASCII Art video sink",
-  plugin_init,
-  VERSION,
-  "GPL",
-  GST_PACKAGE,
-  GST_ORIGIN
-)
+GST_PLUGIN_DEFINE (GST_VERSION_MAJOR,
+    GST_VERSION_MINOR,
+    "cacasink",
+    "Colored ASCII Art video sink",
+    plugin_init, VERSION, GST_LICENSE, GST_PACKAGE, GST_ORIGIN)