Move videocrop and osxvideo to -good.

Original commit message from CVS:
* configure.ac:
* docs/plugins/Makefile.am:
* docs/plugins/gst-plugins-bad-plugins-docs.sgml:
* docs/plugins/gst-plugins-bad-plugins-sections.txt:
* docs/plugins/inspect/plugin-osxvideo.xml:
* docs/plugins/inspect/plugin-videocrop.xml:
* gst-plugins-bad.spec.in:
* gst/videocrop/Makefile.am:
* gst/videocrop/gstvideocrop.c:
* gst/videocrop/gstvideocrop.h:
* gst/videocrop/videocrop.vcproj:
* sys/Makefile.am:
* sys/osxvideo/Makefile.am:
* sys/osxvideo/cocoawindow.h:
* sys/osxvideo/cocoawindow.m:
* sys/osxvideo/osxvideosink.h:
* sys/osxvideo/osxvideosink.m:
* tests/check/Makefile.am:
* tests/check/elements/videocrop.c:
* tests/icles/Makefile.am:
* tests/icles/videocrop-test.c:
Move videocrop and osxvideo to -good.

4 files changed, 0 insertions, 974 deletions

diff --git a/gst/videocrop/Makefile.am b/gst/videocrop/Makefile.am
deleted file mode 100644
index 9557742c..00000000
--- a/gst/videocrop/Makefile.am
+++ /dev/null
@@ -1,12 +0,0 @@
-plugin_LTLIBRARIES = libgstvideocrop.la
-
-# Note: we only use defines from gst/video/video.h, but none
-# of the functions, so we don't need to link to libgstvideo
-
-libgstvideocrop_la_SOURCES = gstvideocrop.c
-libgstvideocrop_la_CFLAGS = $(GST_CFLAGS) $(GST_BASE_CFLAGS) \
-	$(GST_PLUGINS_BASE_CFLAGS)
-libgstvideocrop_la_LIBADD = $(GST_BASE_LIBS)
-libgstvideocrop_la_LDFLAGS = $(GST_PLUGIN_LDFLAGS)
-
-noinst_HEADERS = gstvideocrop.h
diff --git a/gst/videocrop/gstvideocrop.c b/gst/videocrop/gstvideocrop.c
deleted file mode 100644
index 78375545..00000000
--- a/gst/videocrop/gstvideocrop.c
+++ /dev/null
@@ -1,725 +0,0 @@
-/* GStreamer video frame cropping
- * Copyright (C) 2006 Tim-Philipp Müller <tim centricular net>
- *
- * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Library General Public
- * License as published by the Free Software Foundation; either
- * version 2 of the License, or (at your option) any later version.
- *
- * This library is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * Library General Public License for more details.
- *
- * You should have received a copy of the GNU Library General Public
- * License along with this library; if not, write to the
- * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
- * Boston, MA 02111-1307, USA.
- */
-
-/**
- * SECTION:element-videocrop
- * @see_also: GstVideoBox
- *
- * <refsect2>
- * <para>
- * This element crops video frames, meaning it can remove parts of the
- * picture on the left, right, top or bottom of the picture and output
- * a smaller picture than the input picture, with the unwanted parts at the
- * border removed.
- * </para>
- * <para>
- * The videocrop element is similar to the videobox element, but its main
- * goal is to support a multitude of formats as efficiently as possible.
- * Unlike videbox, it cannot add borders to the picture and unlike videbox
- * it will always output images in exactly the same format as the input image.
- * </para>
- * <para>
- * If there is nothing to crop, the element will operate in pass-through mode.
- * </para>
- * <para>
- * Note that no special efforts are made to handle chroma-subsampled formats
- * in the case of odd-valued cropping and compensate for sub-unit chroma plane
- * shifts for such formats in the case where the "left" or "top" property is
- * set to an odd number. This doesn't matter for most use cases, but it might
- * matter for yours.
- * </para>
- * <title>Example launch line</title>
- * <para>
- * <programlisting>
- * gst-launch -v videotestsrc ! videocrop top=42 left=1 right=4 bottom=0 ! ximagesink
- * </programlisting>
- * </para>
- * </refsect2>
- */
-
-/* TODO:
- *  - for packed formats, we could avoid memcpy() in case crop_left
- *    and crop_right are 0 and just create a sub-buffer of the input
- *    buffer
- */
-
-#ifdef HAVE_CONFIG_H
-#include "config.h"
-#endif
-
-#include <gst/gst.h>
-#include <gst/video/video.h>
-
-#include "gstvideocrop.h"
-
-#include <string.h>
-
-GST_DEBUG_CATEGORY_STATIC (videocrop_debug);
-#define GST_CAT_DEFAULT videocrop_debug
-
-static const GstElementDetails video_crop_details = GST_ELEMENT_DETAILS ("Crop",
-    "Filter/Effect/Video",
-    "Crops video into a user-defined region",
-    "Tim-Philipp Müller <tim centricular net>");
-
-enum
-{
-  ARG_0,
-  ARG_LEFT,
-  ARG_RIGHT,
-  ARG_TOP,
-  ARG_BOTTOM
-};
-
-/* the formats we support */
-#define VIDEO_CROP_CAPS                          \
-  GST_VIDEO_CAPS_RGBx ";"                        \
-  GST_VIDEO_CAPS_xRGB ";"                        \
-  GST_VIDEO_CAPS_BGRx ";"                        \
-  GST_VIDEO_CAPS_xBGR ";"                        \
-  GST_VIDEO_CAPS_RGBA ";"                        \
-  GST_VIDEO_CAPS_ARGB ";"                        \
-  GST_VIDEO_CAPS_BGRA ";"                        \
-  GST_VIDEO_CAPS_ABGR ";"                        \
-  GST_VIDEO_CAPS_RGB ";"                         \
-  GST_VIDEO_CAPS_BGR ";"                         \
-  GST_VIDEO_CAPS_YUV ("AYUV") ";"                \
-  GST_VIDEO_CAPS_YUV ("YUY2") ";"                \
-  GST_VIDEO_CAPS_YUV ("YVYU") ";"                \
-  GST_VIDEO_CAPS_YUV ("UYVY") ";"                \
-  GST_VIDEO_CAPS_YUV ("Y800") ";"                \
-  GST_VIDEO_CAPS_YUV ("I420") ";"                \
-  GST_VIDEO_CAPS_YUV ("YV12") ";"                \
-  GST_VIDEO_CAPS_RGB_16 ";"                      \
-  GST_VIDEO_CAPS_RGB_15
-
-static GstStaticPadTemplate src_template = GST_STATIC_PAD_TEMPLATE ("src",
-    GST_PAD_SRC,
-    GST_PAD_ALWAYS,
-    GST_STATIC_CAPS (VIDEO_CROP_CAPS)
-    );
-
-static GstStaticPadTemplate sink_template = GST_STATIC_PAD_TEMPLATE ("sink",
-    GST_PAD_SINK,
-    GST_PAD_ALWAYS,
-    GST_STATIC_CAPS (VIDEO_CROP_CAPS)
-    );
-
-GST_BOILERPLATE (GstVideoCrop, gst_video_crop, GstBaseTransform,
-    GST_TYPE_BASE_TRANSFORM);
-
-static void gst_video_crop_set_property (GObject * object, guint prop_id,
-    const GValue * value, GParamSpec * pspec);
-static void gst_video_crop_get_property (GObject * object, guint prop_id,
-    GValue * value, GParamSpec * pspec);
-
-static GstCaps *gst_video_crop_transform_caps (GstBaseTransform * trans,
-    GstPadDirection direction, GstCaps * caps);
-static GstFlowReturn gst_video_crop_transform (GstBaseTransform * trans,
-    GstBuffer * inbuf, GstBuffer * outbuf);
-static gboolean gst_video_crop_get_unit_size (GstBaseTransform * trans,
-    GstCaps * caps, guint * size);
-static gboolean gst_video_crop_set_caps (GstBaseTransform * trans,
-    GstCaps * in_caps, GstCaps * outcaps);
-
-static void
-gst_video_crop_base_init (gpointer g_class)
-{
-  GstElementClass *element_class = GST_ELEMENT_CLASS (g_class);
-
-  gst_element_class_set_details (element_class, &video_crop_details);
-
-  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));
-}
-
-static void
-gst_video_crop_class_init (GstVideoCropClass * klass)
-{
-  GObjectClass *gobject_class;
-  GstBaseTransformClass *basetransform_class;
-
-  gobject_class = (GObjectClass *) klass;
-  basetransform_class = (GstBaseTransformClass *) klass;
-
-  gobject_class->set_property = gst_video_crop_set_property;
-  gobject_class->get_property = gst_video_crop_get_property;
-
-  g_object_class_install_property (gobject_class, ARG_LEFT,
-      g_param_spec_int ("left", "Left", "Pixels to crop at left",
-          0, G_MAXINT, 0, G_PARAM_READWRITE));
-  g_object_class_install_property (gobject_class, ARG_RIGHT,
-      g_param_spec_int ("right", "Right", "Pixels to crop at right",
-          0, G_MAXINT, 0, G_PARAM_READWRITE));
-  g_object_class_install_property (gobject_class, ARG_TOP,
-      g_param_spec_int ("top", "Top", "Pixels to crop at top",
-          0, G_MAXINT, 0, G_PARAM_READWRITE));
-  g_object_class_install_property (gobject_class, ARG_BOTTOM,
-      g_param_spec_int ("bottom", "Bottom", "Pixels to crop at bottom",
-          0, G_MAXINT, 0, G_PARAM_READWRITE));
-
-  basetransform_class->transform = GST_DEBUG_FUNCPTR (gst_video_crop_transform);
-  basetransform_class->transform_caps =
-      GST_DEBUG_FUNCPTR (gst_video_crop_transform_caps);
-  basetransform_class->set_caps = GST_DEBUG_FUNCPTR (gst_video_crop_set_caps);
-  basetransform_class->get_unit_size =
-      GST_DEBUG_FUNCPTR (gst_video_crop_get_unit_size);
-
-  basetransform_class->passthrough_on_same_caps = FALSE;
-}
-
-static void
-gst_video_crop_init (GstVideoCrop * vcrop, GstVideoCropClass * klass)
-{
-  vcrop->crop_right = 0;
-  vcrop->crop_left = 0;
-  vcrop->crop_top = 0;
-  vcrop->crop_bottom = 0;
-  vcrop->noop = TRUE;
-  GST_BASE_TRANSFORM (vcrop)->passthrough = vcrop->noop;
-}
-
-static gboolean
-gst_video_crop_get_image_details_from_caps (GstVideoCrop * vcrop,
-    GstVideoCropImageDetails * details, GstCaps * caps)
-{
-  GstStructure *structure;
-  gint width, height;
-
-  structure = gst_caps_get_structure (caps, 0);
-  if (!gst_structure_get_int (structure, "width", &width) ||
-      !gst_structure_get_int (structure, "height", &height)) {
-    goto incomplete_format;
-  }
-
-  details->width = width;
-  details->height = height;
-
-  if (gst_structure_has_name (structure, "video/x-raw-rgb")) {
-    gint bpp = 0;
-
-    if (!gst_structure_get_int (structure, "bpp", &bpp) || (bpp & 0x07) != 0)
-      goto incomplete_format;
-
-    details->packing = VIDEO_CROP_PIXEL_FORMAT_PACKED_SIMPLE;
-    details->bytes_per_pixel = bpp / 8;
-    details->stride = GST_ROUND_UP_4 (width * details->bytes_per_pixel);
-    details->size = details->stride * height;
-  } else if (gst_structure_has_name (structure, "video/x-raw-yuv")) {
-    guint32 format = 0;
-
-    if (!gst_structure_get_fourcc (structure, "format", &format))
-      goto incomplete_format;
-
-    switch (format) {
-      case GST_MAKE_FOURCC ('A', 'Y', 'U', 'V'):
-        details->packing = VIDEO_CROP_PIXEL_FORMAT_PACKED_SIMPLE;
-        details->bytes_per_pixel = 4;
-        details->stride = GST_ROUND_UP_4 (width * 4);
-        details->size = details->stride * height;
-        break;
-      case GST_MAKE_FOURCC ('Y', 'V', 'Y', 'U'):
-      case GST_MAKE_FOURCC ('Y', 'U', 'Y', '2'):
-      case GST_MAKE_FOURCC ('U', 'Y', 'V', 'Y'):
-        details->packing = VIDEO_CROP_PIXEL_FORMAT_PACKED_COMPLEX;
-        details->bytes_per_pixel = 2;
-        details->stride = GST_ROUND_UP_4 (width * 2);
-        details->size = details->stride * height;
-        if (format == GST_MAKE_FOURCC ('U', 'Y', 'V', 'Y')) {
-          /* UYVY = 4:2:2 - [U0 Y0 V0 Y1] [U2 Y2 V2 Y3] [U4 Y4 V4 Y5] */
-          details->macro_y_off = 1;
-        } else {
-          /* YUYV = 4:2:2 - [Y0 U0 Y1 V0] [Y2 U2 Y3 V2] [Y4 U4 Y5 V4] = YUY2 */
-          details->macro_y_off = 0;
-        }
-        break;
-      case GST_MAKE_FOURCC ('Y', '8', '0', '0'):
-        details->packing = VIDEO_CROP_PIXEL_FORMAT_PACKED_SIMPLE;
-        details->bytes_per_pixel = 1;
-        details->stride = GST_ROUND_UP_4 (width);
-        details->size = details->stride * height;
-        break;
-      case GST_MAKE_FOURCC ('I', '4', '2', '0'):
-      case GST_MAKE_FOURCC ('Y', 'V', '1', '2'):{
-        details->packing = VIDEO_CROP_PIXEL_FORMAT_PLANAR;
-
-        details->y_stride = GST_ROUND_UP_4 (width);
-        details->u_stride = GST_ROUND_UP_8 (width) / 2;
-        details->v_stride = GST_ROUND_UP_8 (width) / 2;
-
-        /* I420 and YV12 have U/V planes swapped, but doesn't matter for us */
-        details->y_off = 0;
-        details->u_off = 0 + details->y_stride * GST_ROUND_UP_2 (height);
-        details->v_off = details->u_off +
-            details->u_stride * (GST_ROUND_UP_2 (height) / 2);
-        details->size = details->v_off +
-            details->v_stride * (GST_ROUND_UP_2 (height) / 2);
-        break;
-      }
-      default:
-        goto unknown_format;
-    }
-  } else {
-    goto unknown_format;
-  }
-
-  return TRUE;
-
-  /* ERRORS */
-unknown_format:
-  {
-    GST_ELEMENT_ERROR (vcrop, STREAM, NOT_IMPLEMENTED, (NULL),
-        ("Unsupported format"));
-    return FALSE;
-  }
-
-incomplete_format:
-  {
-    GST_ELEMENT_ERROR (vcrop, CORE, NEGOTIATION, (NULL),
-        ("Incomplete caps, some required field is missing"));
-    return FALSE;
-  }
-}
-
-static gboolean
-gst_video_crop_get_unit_size (GstBaseTransform * trans, GstCaps * caps,
-    guint * size)
-{
-  GstVideoCropImageDetails img_details = { 0, };
-  GstVideoCrop *vcrop = GST_VIDEO_CROP (trans);
-
-  if (!gst_video_crop_get_image_details_from_caps (vcrop, &img_details, caps))
-    return FALSE;
-
-  *size = img_details.size;
-  return TRUE;
-}
-
-#define ROUND_DOWN_2(n)  ((n)&(~1))
-
-static void
-gst_video_crop_transform_packed_complex (GstVideoCrop * vcrop,
-    GstBuffer * inbuf, GstBuffer * outbuf)
-{
-  guint8 *in_data, *out_data;
-  guint i, dx;
-
-  in_data = GST_BUFFER_DATA (inbuf);
-  out_data = GST_BUFFER_DATA (outbuf);
-
-  in_data += vcrop->crop_top * vcrop->in.stride;
-
-  /* rounding down here so we end up at the start of a macro-pixel and not
-   * in the middle of one */
-  in_data += ROUND_DOWN_2 (vcrop->crop_left) * vcrop->in.bytes_per_pixel;
-
-  dx = vcrop->out.width * vcrop->out.bytes_per_pixel;
-
-  /* UYVY = 4:2:2 - [U0 Y0 V0 Y1] [U2 Y2 V2 Y3] [U4 Y4 V4 Y5]
-   * YUYV = 4:2:2 - [Y0 U0 Y1 V0] [Y2 U2 Y3 V2] [Y4 U4 Y5 V4] = YUY2 */
-  if ((vcrop->crop_left % 2) != 0) {
-    for (i = 0; i < vcrop->out.height; ++i) {
-      gint j;
-
-      memcpy (out_data, in_data, dx);
-
-      /* move just the Y samples one pixel to the left, don't worry about
-       * chroma shift */
-      for (j = vcrop->in.macro_y_off; j < vcrop->out.stride - 2; j += 2)
-        out_data[j] = in_data[j + 2];
-
-      in_data += vcrop->in.stride;
-      out_data += vcrop->out.stride;
-    }
-  } else {
-    for (i = 0; i < vcrop->out.height; ++i) {
-      memcpy (out_data, in_data, dx);
-      in_data += vcrop->in.stride;
-      out_data += vcrop->out.stride;
-    }
-  }
-}
-
-static void
-gst_video_crop_transform_packed_simple (GstVideoCrop * vcrop,
-    GstBuffer * inbuf, GstBuffer * outbuf)
-{
-  guint8 *in_data, *out_data;
-  guint i, dx;
-
-  in_data = GST_BUFFER_DATA (inbuf);
-  out_data = GST_BUFFER_DATA (outbuf);
-
-  in_data += vcrop->crop_top * vcrop->in.stride;
-  in_data += vcrop->crop_left * vcrop->in.bytes_per_pixel;
-
-  dx = vcrop->out.width * vcrop->out.bytes_per_pixel;
-
-  for (i = 0; i < vcrop->out.height; ++i) {
-    memcpy (out_data, in_data, dx);
-    in_data += vcrop->in.stride;
-    out_data += vcrop->out.stride;
-  }
-}
-
-static void
-gst_video_crop_transform_planar (GstVideoCrop * vcrop, GstBuffer * inbuf,
-    GstBuffer * outbuf)
-{
-  guint8 *y_out, *u_out, *v_out;
-  guint8 *y_in, *u_in, *v_in;
-  guint i, dx;
-
-  /* Y plane */
-  y_in = GST_BUFFER_DATA (inbuf);
-  y_out = GST_BUFFER_DATA (outbuf);
-
-  y_in += (vcrop->crop_top * vcrop->in.y_stride) + vcrop->crop_left;
-  dx = vcrop->out.width * 1;
-
-  for (i = 0; i < vcrop->out.height; ++i) {
-    memcpy (y_out, y_in, dx);
-    y_in += vcrop->in.y_stride;
-    y_out += vcrop->out.y_stride;
-  }
-
-  /* U + V planes */
-  u_in = GST_BUFFER_DATA (inbuf) + vcrop->in.u_off;
-  u_out = GST_BUFFER_DATA (outbuf) + vcrop->out.u_off;
-
-  u_in += (vcrop->crop_top / 2) * vcrop->in.u_stride;
-  u_in += vcrop->crop_left / 2;
-
-  v_in = GST_BUFFER_DATA (inbuf) + vcrop->in.v_off;
-  v_out = GST_BUFFER_DATA (outbuf) + vcrop->out.v_off;
-
-  v_in += (vcrop->crop_top / 2) * vcrop->in.v_stride;
-  v_in += vcrop->crop_left / 2;
-
-  dx = GST_ROUND_UP_2 (vcrop->out.width) / 2;
-
-  for (i = 0; i < GST_ROUND_UP_2 (vcrop->out.height) / 2; ++i) {
-    memcpy (u_out, u_in, dx);
-    memcpy (v_out, v_in, dx);
-    u_in += vcrop->in.u_stride;
-    u_out += vcrop->out.u_stride;
-    v_in += vcrop->in.v_stride;
-    v_out += vcrop->out.v_stride;
-  }
-}
-
-static GstFlowReturn
-gst_video_crop_transform (GstBaseTransform * trans, GstBuffer * inbuf,
-    GstBuffer * outbuf)
-{
-  GstVideoCrop *vcrop = GST_VIDEO_CROP (trans);
-
-  /* we should be operating in passthrough mode if there's nothing to do */
-  g_assert (vcrop->noop == FALSE);
-
-  GST_OBJECT_LOCK (vcrop);
-
-  if (G_UNLIKELY ((vcrop->crop_left + vcrop->crop_right) >= vcrop->in.width ||
-          (vcrop->crop_top + vcrop->crop_bottom) >= vcrop->in.height)) {
-    GST_OBJECT_UNLOCK (vcrop);
-    goto cropping_too_much;
-  }
-
-  switch (vcrop->in.packing) {
-    case VIDEO_CROP_PIXEL_FORMAT_PACKED_SIMPLE:
-      gst_video_crop_transform_packed_simple (vcrop, inbuf, outbuf);
-      break;
-    case VIDEO_CROP_PIXEL_FORMAT_PACKED_COMPLEX:
-      gst_video_crop_transform_packed_complex (vcrop, inbuf, outbuf);
-      break;
-    case VIDEO_CROP_PIXEL_FORMAT_PLANAR:
-      gst_video_crop_transform_planar (vcrop, inbuf, outbuf);
-      break;
-    default:
-      g_assert_not_reached ();
-  }
-
-  GST_OBJECT_UNLOCK (vcrop);
-
-  return GST_FLOW_OK;
-
-cropping_too_much:
-  {
-    /* is there a better error code for this? */
-    GST_ELEMENT_ERROR (vcrop, LIBRARY, SETTINGS, (NULL),
-        ("Can't crop more pixels than there are"));
-    return GST_FLOW_ERROR;
-  }
-}
-
-static gint
-gst_video_crop_transform_dimension (gint val, gint delta)
-{
-  gint64 new_val = (gint64) val + (gint64) delta;
-
-  new_val = CLAMP (new_val, 1, G_MAXINT);
-
-  return (gint) new_val;
-}
-
-static gboolean
-gst_video_crop_transform_dimension_value (const GValue * src_val,
-    gint delta, GValue * dest_val)
-{
-  gboolean ret = TRUE;
-
-  g_value_init (dest_val, G_VALUE_TYPE (src_val));
-
-  if (G_VALUE_HOLDS_INT (src_val)) {
-    gint ival = g_value_get_int (src_val);
-
-    ival = gst_video_crop_transform_dimension (ival, delta);
-    g_value_set_int (dest_val, ival);
-  } else if (GST_VALUE_HOLDS_INT_RANGE (src_val)) {
-    gint min = gst_value_get_int_range_min (src_val);
-    gint max = gst_value_get_int_range_max (src_val);
-
-    min = gst_video_crop_transform_dimension (min, delta);
-    max = gst_video_crop_transform_dimension (max, delta);
-    gst_value_set_int_range (dest_val, min, max);
-  } else if (GST_VALUE_HOLDS_LIST (src_val)) {
-    gint i;
-
-    for (i = 0; i < gst_value_list_get_size (src_val); ++i) {
-      const GValue *list_val;
-      GValue newval = { 0, };
-
-      list_val = gst_value_list_get_value (src_val, i);
-      if (gst_video_crop_transform_dimension_value (list_val, delta, &newval))
-        gst_value_list_append_value (dest_val, &newval);
-      g_value_unset (&newval);
-    }
-
-    if (gst_value_list_get_size (dest_val) == 0) {
-      g_value_unset (dest_val);
-      ret = FALSE;
-    }
-  } else {
-    g_value_unset (dest_val);
-    ret = FALSE;
-  }
-
-  return ret;
-}
-
-static GstCaps *
-gst_video_crop_transform_caps (GstBaseTransform * trans,
-    GstPadDirection direction, GstCaps * caps)
-{
-  GstVideoCrop *vcrop;
-  GstCaps *other_caps;
-  gint dy, dx, i;
-
-  vcrop = GST_VIDEO_CROP (trans);
-
-  GST_OBJECT_LOCK (vcrop);
-
-  GST_LOG_OBJECT (vcrop, "l=%d,r=%d,b=%d,t=%d noop=%d",
-      vcrop->crop_left, vcrop->crop_right, vcrop->crop_bottom,
-      vcrop->crop_top, vcrop->noop);
-
-  if (vcrop->noop) {
-    GST_OBJECT_UNLOCK (vcrop);
-    return gst_caps_ref (caps);
-  }
-
-  if (direction == GST_PAD_SRC) {
-    dx = vcrop->crop_left + vcrop->crop_right;
-    dy = vcrop->crop_top + vcrop->crop_bottom;
-  } else {
-    dx = 0 - (vcrop->crop_left + vcrop->crop_right);
-    dy = 0 - (vcrop->crop_top + vcrop->crop_bottom);
-  }
-  GST_OBJECT_UNLOCK (vcrop);
-
-  GST_LOG_OBJECT (vcrop, "transforming caps %" GST_PTR_FORMAT, caps);
-
-  other_caps = gst_caps_new_empty ();
-
-  for (i = 0; i < gst_caps_get_size (caps); ++i) {
-    const GValue *v;
-    GstStructure *structure, *new_structure;
-    GValue w_val = { 0, }, h_val = {
-    0,};
-
-    structure = gst_caps_get_structure (caps, i);
-
-    v = gst_structure_get_value (structure, "width");
-    if (!gst_video_crop_transform_dimension_value (v, dx, &w_val)) {
-      GST_WARNING_OBJECT (vcrop, "could not tranform width value with dx=%d"
-          ", caps structure=%" GST_PTR_FORMAT, dx, structure);
-      continue;
-    }
-
-    v = gst_structure_get_value (structure, "height");
-    if (!gst_video_crop_transform_dimension_value (v, dy, &h_val)) {
-      g_value_unset (&w_val);
-      GST_WARNING_OBJECT (vcrop, "could not tranform height value with dy=%d"
-          ", caps structure=%" GST_PTR_FORMAT, dy, structure);
-      continue;
-    }
-
-    new_structure = gst_structure_copy (structure);
-    gst_structure_set_value (new_structure, "width", &w_val);
-    gst_structure_set_value (new_structure, "height", &h_val);
-    g_value_unset (&w_val);
-    g_value_unset (&h_val);
-    GST_LOG_OBJECT (vcrop, "transformed structure %2d: %" GST_PTR_FORMAT
-        " => %" GST_PTR_FORMAT, i, structure, new_structure);
-    gst_caps_append_structure (other_caps, new_structure);
-  }
-
-  if (gst_caps_is_empty (other_caps)) {
-    gst_caps_unref (other_caps);
-    other_caps = NULL;
-  }
-
-  return other_caps;
-}
-
-static gboolean
-gst_video_crop_set_caps (GstBaseTransform * trans, GstCaps * incaps,
-    GstCaps * outcaps)
-{
-  GstVideoCrop *crop = GST_VIDEO_CROP (trans);
-
-  if (!gst_video_crop_get_image_details_from_caps (crop, &crop->in, incaps)) {
-    GST_DEBUG_OBJECT (crop, "failed to parse input caps %" GST_PTR_FORMAT,
-        incaps);
-    return FALSE;
-  }
-
-  if (!gst_video_crop_get_image_details_from_caps (crop, &crop->out, outcaps)) {
-    GST_DEBUG_OBJECT (crop, "failed to parse output caps %" GST_PTR_FORMAT,
-        outcaps);
-    return FALSE;
-  }
-
-  GST_LOG_OBJECT (crop, "incaps = %" GST_PTR_FORMAT ", outcaps = %"
-      GST_PTR_FORMAT, incaps, outcaps);
-
-  return TRUE;
-}
-
-/* This is extremely hackish, but the only way to force basetransform to
- * renegotiated at the moment. There should really be a basetransform
- * function for this */
-static void
-gst_videocrop_clear_negotiated_caps_locked (GstVideoCrop * crop)
-{
-  GST_LOG_OBJECT (crop, "clearing negotiated caps");
-  GST_BASE_TRANSFORM (crop)->negotiated = FALSE;
-  gst_caps_replace (&GST_PAD_CAPS (GST_BASE_TRANSFORM (crop)->srcpad), NULL);
-  gst_caps_replace (&GST_PAD_CAPS (GST_BASE_TRANSFORM (crop)->sinkpad), NULL);
-  gst_caps_replace (&GST_BASE_TRANSFORM (crop)->cache_caps1, NULL);
-  GST_BASE_TRANSFORM (crop)->cache_caps1_size = 0;
-  gst_caps_replace (&GST_BASE_TRANSFORM (crop)->cache_caps2, NULL);
-  GST_BASE_TRANSFORM (crop)->cache_caps2_size = 0;
-  GST_LOG_OBJECT (crop, "clearing caps done");
-}
-
-static void
-gst_video_crop_set_property (GObject * object, guint prop_id,
-    const GValue * value, GParamSpec * pspec)
-{
-  GstVideoCrop *video_crop;
-
-  video_crop = GST_VIDEO_CROP (object);
-
-  GST_OBJECT_LOCK (video_crop);
-  switch (prop_id) {
-    case ARG_LEFT:
-      video_crop->crop_left = g_value_get_int (value);
-      break;
-    case ARG_RIGHT:
-      video_crop->crop_right = g_value_get_int (value);
-      break;
-    case ARG_TOP:
-      video_crop->crop_top = g_value_get_int (value);
-      break;
-    case ARG_BOTTOM:
-      video_crop->crop_bottom = g_value_get_int (value);
-      break;
-    default:
-      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
-      break;
-  }
-
-  video_crop->noop = ((video_crop->crop_left | video_crop->crop_right |
-          video_crop->crop_top | video_crop->crop_bottom) == 0);
-
-  GST_LOG_OBJECT (video_crop, "l=%d,r=%d,b=%d,t=%d noop=%d",
-      video_crop->crop_left, video_crop->crop_right, video_crop->crop_bottom,
-      video_crop->crop_top, video_crop->noop);
-
-  GST_BASE_TRANSFORM (video_crop)->passthrough = video_crop->noop;
-  gst_videocrop_clear_negotiated_caps_locked (video_crop);
-  GST_OBJECT_UNLOCK (video_crop);
-}
-
-static void
-gst_video_crop_get_property (GObject * object, guint prop_id, GValue * value,
-    GParamSpec * pspec)
-{
-  GstVideoCrop *video_crop;
-
-  video_crop = GST_VIDEO_CROP (object);
-
-  GST_OBJECT_LOCK (video_crop);
-  switch (prop_id) {
-    case ARG_LEFT:
-      g_value_set_int (value, video_crop->crop_left);
-      break;
-    case ARG_RIGHT:
-      g_value_set_int (value, video_crop->crop_right);
-      break;
-    case ARG_TOP:
-      g_value_set_int (value, video_crop->crop_top);
-      break;
-    case ARG_BOTTOM:
-      g_value_set_int (value, video_crop->crop_bottom);
-      break;
-    default:
-      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
-      break;
-  }
-  GST_OBJECT_UNLOCK (video_crop);
-}
-
-static gboolean
-plugin_init (GstPlugin * plugin)
-{
-  GST_DEBUG_CATEGORY_INIT (videocrop_debug, "videocrop", 0, "videocrop");
-
-  return gst_element_register (plugin, "videocrop", GST_RANK_NONE,
-      GST_TYPE_VIDEO_CROP);
-}
-
-GST_PLUGIN_DEFINE (GST_VERSION_MAJOR,
-    GST_VERSION_MINOR,
-    "videocrop",
-    "Crops video into a user-defined region",
-    plugin_init, VERSION, GST_LICENSE, GST_PACKAGE_NAME, GST_PACKAGE_ORIGIN)
diff --git a/gst/videocrop/gstvideocrop.h b/gst/videocrop/gstvideocrop.h
deleted file mode 100644
index 0075997f..00000000
--- a/gst/videocrop/gstvideocrop.h
+++ /dev/null
@@ -1,92 +0,0 @@
-/* GStreamer video frame cropping
- * Copyright (C) 2006 Tim-Philipp Müller <tim centricular net>
- *
- * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Library General Public
- * License as published by the Free Software Foundation; either
- * version 2 of the License, or (at your option) any later version.
- *
- * This library is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * Library General Public License for more details.
- *
- * You should have received a copy of the GNU Library General Public
- * License along with this library; if not, write to the
- * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
- * Boston, MA 02111-1307, USA.
- */
-
-#ifndef __GST_VIDEO_CROP_H__
-#define __GST_VIDEO_CROP_H__
-
-#include <gst/base/gstbasetransform.h>
-
-G_BEGIN_DECLS
-
-#define GST_TYPE_VIDEO_CROP \
-  (gst_video_crop_get_type())
-#define GST_VIDEO_CROP(obj) \
-  (G_TYPE_CHECK_INSTANCE_CAST((obj),GST_TYPE_VIDEO_CROP,GstVideoCrop))
-#define GST_VIDEO_CROP_CLASS(klass) \
-  (G_TYPE_CHECK_CLASS_CAST((klass),GST_TYPE_VIDEO_CROP,GstVideoCropClass))
-#define GST_IS_VIDEO_CROP(obj) \
-  (G_TYPE_CHECK_INSTANCE_TYPE((obj),GST_TYPE_VIDEO_CROP))
-#define GST_IS_VIDEO_CROP_CLASS(klass) \
-  (G_TYPE_CHECK_CLASS_TYPE((klass),GST_TYPE_VIDEO_CROP))
-
-typedef enum {
-  VIDEO_CROP_PIXEL_FORMAT_PACKED_SIMPLE = 0,  /* RGBx, AYUV */
-  VIDEO_CROP_PIXEL_FORMAT_PACKED_COMPLEX,     /* UYVY, YVYU */
-  VIDEO_CROP_PIXEL_FORMAT_PLANAR              /* I420, YV12 */
-} VideoCropPixelFormat;
-
-typedef struct _GstVideoCropImageDetails GstVideoCropImageDetails;
-struct _GstVideoCropImageDetails
-{
-  /*< private >*/
-  VideoCropPixelFormat  packing;
-
-  guint width;
-  guint height;
-  guint size;
-
-  /* for packed RGB and YUV */
-  guint   stride;
-  guint   bytes_per_pixel;
-  guint8  macro_y_off;            /* for YUY2, YVYU, UYVY, Y offset within macropixel in bytes */
-
-  /* for planar YUV */
-  guint y_stride, y_off;
-  guint u_stride, u_off;
-  guint v_stride, v_off;
-};
-
-typedef struct _GstVideoCrop GstVideoCrop;
-typedef struct _GstVideoCropClass GstVideoCropClass;
-
-struct _GstVideoCrop
-{
-  GstBaseTransform basetransform;
-
-  /*< private >*/
-  gboolean noop;                /* TRUE if crop_left,_right,_top and _bottom are all 0 */
-
-  gint crop_left;
-  gint crop_right;
-  gint crop_top;
-  gint crop_bottom;
-
-  GstVideoCropImageDetails in;  /* details of input image */
-  GstVideoCropImageDetails out; /* details of output image */
-};
-
-struct _GstVideoCropClass
-{
-  GstBaseTransformClass basetransform_class;
-};
-
-G_END_DECLS
-
-#endif /* __GST_VIDEO_CROP_H__ */
-
diff --git a/gst/videocrop/videocrop.vcproj b/gst/videocrop/videocrop.vcproj
deleted file mode 100644
index aa81b925..00000000
--- a/gst/videocrop/videocrop.vcproj
+++ /dev/null
@@ -1,145 +0,0 @@
-<?xml version="1.0" encoding="Windows-1252"?>
-<VisualStudioProject
-	ProjectType="Visual C++"
-	Version="7.10"
-	Name="videocrop"
-	ProjectGUID="{979C216F-0ACF-4956-AE00-055A42D678D5}"
-	Keyword="Win32Proj">
-	<Platforms>
-		<Platform
-			Name="Win32"/>
-	</Platforms>
-	<Configurations>
-		<Configuration
-			Name="Debug|Win32"
-			OutputDirectory="../../win32/Debug"
-			IntermediateDirectory="../../win32/Debug"
-			ConfigurationType="2"
-			CharacterSet="2">
-			<Tool
-				Name="VCCLCompilerTool"
-				Optimization="0"
-				AdditionalIncludeDirectories="../../../gstreamer/win32;../../../gstreamer;../../../gstreamer/libs;../../../glib;../../../glib/glib;../../../glib/gmodule;&quot;../../gst-libs&quot;;../../../popt/include;../../../libxml2/include/libxml2"
-				PreprocessorDefinitions="WIN32;_DEBUG;_WINDOWS;_USRDLL;videocrop_EXPORTS;HAVE_CONFIG_H;_USE_MATH_DEFINES"
-				MinimalRebuild="TRUE"
-				BasicRuntimeChecks="3"
-				RuntimeLibrary="3"
-				UsePrecompiledHeader="0"
-				WarningLevel="3"
-				Detect64BitPortabilityProblems="TRUE"
-				DebugInformationFormat="4"/>
-			<Tool
-				Name="VCCustomBuildTool"/>
-			<Tool
-				Name="VCLinkerTool"
-				AdditionalDependencies="glib-2.0.lib gmodule-2.0.lib gthread-2.0.lib gobject-2.0.lib libgstreamer.lib gstbytestream.lib iconv.lib intl.lib"
-				OutputFile="$(OutDir)/gstvideocrop.dll"
-				LinkIncremental="2"
-				AdditionalLibraryDirectories="../../../gstreamer/win32/Debug;../../../glib/glib;../../../glib/gmodule;../../../glib/gthread;../../../glib/gobject;../../../gettext/lib;../../../libiconv/lib"
-				ModuleDefinitionFile=""
-				GenerateDebugInformation="TRUE"
-				ProgramDatabaseFile="$(OutDir)/videocrop.pdb"
-				SubSystem="2"
-				OptimizeReferences="2"
-				ImportLibrary="$(OutDir)/gstvideocrop.lib"
-				TargetMachine="1"/>
-			<Tool
-				Name="VCMIDLTool"/>
-			<Tool
-				Name="VCPostBuildEventTool"
-				CommandLine="copy /Y $(TargetPath) c:\gstreamer\plugins"/>
-			<Tool
-				Name="VCPreBuildEventTool"/>
-			<Tool
-				Name="VCPreLinkEventTool"/>
-			<Tool
-				Name="VCResourceCompilerTool"/>
-			<Tool
-				Name="VCWebServiceProxyGeneratorTool"/>
-			<Tool
-				Name="VCXMLDataGeneratorTool"/>
-			<Tool
-				Name="VCWebDeploymentTool"/>
-			<Tool
-				Name="VCManagedWrapperGeneratorTool"/>
-			<Tool
-				Name="VCAuxiliaryManagedWrapperGeneratorTool"/>
-		</Configuration>
-		<Configuration
-			Name="Release|Win32"
-			OutputDirectory="../../win32/Release"
-			IntermediateDirectory="../../win32/Release"
-			ConfigurationType="2"
-			CharacterSet="2">
-			<Tool
-				Name="VCCLCompilerTool"
-				AdditionalIncludeDirectories="../../../gstreamer/win32;../../../gstreamer;../../../gstreamer/libs;../../../glib;../../../glib/glib;../../../glib/gmodule;&quot;../../gst-libs&quot;;../../../popt/include;../../../libxml2/include/libxml2"
-				PreprocessorDefinitions="WIN32;NDEBUG;GST_DISABLE_GST_DEBUG;_WINDOWS;_USRDLL;videocrop_EXPORTS;HAVE_CONFIG_H;_USE_MATH_DEFINES"
-				RuntimeLibrary="2"
-				UsePrecompiledHeader="0"
-				WarningLevel="3"
-				Detect64BitPortabilityProblems="TRUE"
-				DebugInformationFormat="3"/>
-			<Tool
-				Name="VCCustomBuildTool"/>
-			<Tool
-				Name="VCLinkerTool"
-				AdditionalDependencies="glib-2.0.lib gmodule-2.0.lib gthread-2.0.lib gobject-2.0.lib libgstreamer.lib gstbytestream.lib iconv.lib intl.lib"
-				OutputFile="$(OutDir)/gstvideocrop.dll"
-				LinkIncremental="1"
-				AdditionalLibraryDirectories="../../../gstreamer/win32/Release;../../../glib/glib;../../../glib/gmodule;../../../glib/gthread;../../../glib/gobject;../../../gettext/lib;../../../libiconv/lib"
-				ModuleDefinitionFile=""
-				GenerateDebugInformation="TRUE"
-				SubSystem="2"
-				OptimizeReferences="2"
-				EnableCOMDATFolding="2"
-				ImportLibrary="$(OutDir)/gstvideocrop.lib"
-				TargetMachine="1"/>
-			<Tool
-				Name="VCMIDLTool"/>
-			<Tool
-				Name="VCPostBuildEventTool"
-				CommandLine="copy /Y $(TargetPath) c:\gstreamer\plugins"/>
-			<Tool
-				Name="VCPreBuildEventTool"/>
-			<Tool
-				Name="VCPreLinkEventTool"/>
-			<Tool
-				Name="VCResourceCompilerTool"/>
-			<Tool
-				Name="VCWebServiceProxyGeneratorTool"/>
-			<Tool
-				Name="VCXMLDataGeneratorTool"/>
-			<Tool
-				Name="VCWebDeploymentTool"/>
-			<Tool
-				Name="VCManagedWrapperGeneratorTool"/>
-			<Tool
-				Name="VCAuxiliaryManagedWrapperGeneratorTool"/>
-		</Configuration>
-	</Configurations>
-	<References>
-	</References>
-	<Files>
-		<Filter
-			Name="Source Files"
-			Filter="cpp;c;cxx;def;odl;idl;hpj;bat;asm;asmx"
-			UniqueIdentifier="{4FC737F1-C7A5-4376-A066-2A32D752A2FF}">
-			<File
-				RelativePath=".\gstvideocrop.c">
-			</File>
-		</Filter>
-		<Filter
-			Name="Header Files"
-			Filter="h;hpp;hxx;hm;inl;inc;xsd"
-			UniqueIdentifier="{93995380-89BD-4b04-88EB-625FBE52EBFB}">
-		</Filter>
-		<Filter
-			Name="Resource Files"
-			Filter="rc;ico;cur;bmp;dlg;rc2;rct;bin;rgs;gif;jpg;jpeg;jpe;resx"
-			UniqueIdentifier="{67DA6AB6-F800-4c08-8B7A-83BB121AAD01}">
-		</Filter>
-	</Files>
-	<Globals>
-	</Globals>
-</VisualStudioProject>