Faac (AAC Encoder) ported for GStreamer 0.9CHANGELOG_START

Original commit message from CVS:
Faac (AAC Encoder) ported for GStreamer 0.9

6 files changed, 67 insertions, 55 deletions

diff --git a/ChangeLog b/ChangeLog
index 89f77de95..b35478961 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2005-08-31  Flavio Oliveira  <flavio.oliveira@indt.org.br>
+
+	* configure.ac:
+	* ext/Makefile.am:
+	* ext/faac/Makefile.am:
+	* ext/faac/gstfaac.c: Ported to GStreamer 0.9.
+
 2005-08-31  Thomas Vander Stichele  <thomas at apestaart dot org>
 
 	* configure.ac:
diff --git a/common b/common
-Subproject 74223ba3ec3be64622ac71d682b36c1f8f01350
+Subproject b0ee0e4262014001faceb47d71c3a44c75ab86b
diff --git a/configure.ac b/configure.ac
index 64b9741e1..8c0e9fa7e 100644
--- a/configure.ac
+++ b/configure.ac
@@ -322,6 +322,14 @@ dnl ###########################
 dnl # Configure external libs #
 dnl ###########################
 
+dnl **** Free AAC Encoder (FAAC) ****
+translit(dnm, m, l) AM_CONDITIONAL(USE_FAAC, true)
+GST_CHECK_FEATURE(FAAC, [AAC encoder plug-in], faac, [
+  GST_CHECK_LIBHEADER(FAAC, faac, faacEncOpen, -lm, faac.h, FAAC_LIBS="-lfaac -lm")
+  AS_SCRUB_INCLUDE(FAAC_CFLAGS)
+  AC_SUBST(FAAC_LIBS)
+])
+
 dnl **** Free AAC Decoder (FAAD) ****
 translit(dnm, m, l) AM_CONDITIONAL(USE_FAAD, true)
 GST_CHECK_FEATURE(FAAD, [AAC decoder plug-in], faad, [
@@ -414,6 +422,7 @@ gst/Makefile
 gst/qtdemux/Makefile
 sys/Makefile
 ext/Makefile
+ext/faac/Makefile
 ext/faad/Makefile
 common/Makefile
 common/m4/Makefile
diff --git a/ext/Makefile.am b/ext/Makefile.am
index 9f3ca07b9..b2a039eb2 100644
--- a/ext/Makefile.am
+++ b/ext/Makefile.am
@@ -58,11 +58,11 @@ DIVX_DIR=
 DTS_DIR=
 # endif
 
-# if USE_FAAC
-# FAAC_DIR=faac
-# else
+if USE_FAAC
+FAAC_DIR=faac
+else
 FAAC_DIR=
-# endif
+endif
 
 if USE_FAAD
  FAAD_DIR=faad
@@ -259,6 +259,7 @@ SUBDIRS=\
 
 DIST_SUBDIRS=\
 	amrnb \
+	faac \
 	faad \
 	lame \
 	mad \
diff --git a/ext/faac/Makefile.am b/ext/faac/Makefile.am
index c5610e979..d93dd7beb 100644
--- a/ext/faac/Makefile.am
+++ b/ext/faac/Makefile.am
@@ -1,8 +1,9 @@
 plugin_LTLIBRARIES = libgstfaac.la
 
 libgstfaac_la_SOURCES = gstfaac.c
-libgstfaac_la_CFLAGS = $(FAAC_CFLAGS) $(GST_CFLAGS)
-libgstfaac_la_LIBADD = $(FAAC_LIBS) 
+libgstfaac_la_CFLAGS = $(FAAC_CFLAGS) $(GST_CFLAGS) $(GST_PLUGINS_BASE_CFLAGS)
+libgstfaac_la_LIBADD = $(FAAC_LIBS) $(GST_PLUGINS_BASE_LIBS) \
+	-lgstaudio-@GST_MAJORMINOR@
 libgstfaac_la_LDFLAGS = $(GST_PLUGIN_LDFLAGS)
 
 noinst_HEADERS = gstfaac.h
diff --git a/ext/faac/gstfaac.c b/ext/faac/gstfaac.c
index c51487732..233021e83 100644
--- a/ext/faac/gstfaac.c
+++ b/ext/faac/gstfaac.c
@@ -59,11 +59,9 @@ static void gst_faac_set_property (GObject * object,
 static void gst_faac_get_property (GObject * object,
     guint prop_id, GValue * value, GParamSpec * pspec);
 
-static GstPadLinkReturn
-gst_faac_sinkconnect (GstPad * pad, const GstCaps * caps);
-static GstPadLinkReturn
-gst_faac_srcconnect (GstPad * pad, const GstCaps * caps);
-static void gst_faac_chain (GstPad * pad, GstData * data);
+static gboolean gst_faac_sink_setcaps (GstPad * pad, GstCaps * caps);
+static gboolean gst_faac_src_setcaps (GstPad * pad, GstCaps * caps);
+static GstFlowReturn gst_faac_chain (GstPad * pad, GstBuffer * data);
 static GstElementStateReturn gst_faac_change_state (GstElement * element);
 
 static GstElementClass *parent_class = NULL;
@@ -185,6 +183,9 @@ gst_faac_class_init (GstFaacClass * klass)
 
   parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
 
+  gobject_class->set_property = gst_faac_set_property;
+  gobject_class->get_property = gst_faac_get_property;
+
   /* properties */
   g_object_class_install_property (gobject_class, ARG_BITRATE,
       g_param_spec_int ("bitrate", "Bitrate (bps)", "Bitrate in bits/sec",
@@ -209,9 +210,6 @@ gst_faac_class_init (GstFaacClass * klass)
 
   /* virtual functions */
   gstelement_class->change_state = gst_faac_change_state;
-
-  gobject_class->set_property = gst_faac_set_property;
-  gobject_class->get_property = gst_faac_get_property;
 }
 
 static void
@@ -224,20 +222,18 @@ gst_faac_init (GstFaac * faac)
   faac->cache_time = GST_CLOCK_TIME_NONE;
   faac->cache_duration = 0;
 
-  GST_FLAG_SET (faac, GST_ELEMENT_EVENT_AWARE);
-
   faac->sinkpad =
       gst_pad_new_from_template (gst_static_pad_template_get (&sink_template),
       "sink");
-  gst_element_add_pad (GST_ELEMENT (faac), faac->sinkpad);
   gst_pad_set_chain_function (faac->sinkpad, gst_faac_chain);
-  gst_pad_set_link_function (faac->sinkpad, gst_faac_sinkconnect);
+  gst_pad_set_setcaps_function (faac->sinkpad, gst_faac_sink_setcaps);
+  gst_element_add_pad (GST_ELEMENT (faac), faac->sinkpad);
 
   faac->srcpad =
       gst_pad_new_from_template (gst_static_pad_template_get (&src_template),
       "src");
+  gst_pad_set_setcaps_function (faac->srcpad, gst_faac_src_setcaps);
   gst_element_add_pad (GST_ELEMENT (faac), faac->srcpad);
-  gst_pad_set_link_function (faac->srcpad, gst_faac_srcconnect);
 
   /* default properties */
   faac->bitrate = 1024 * 128;
@@ -248,8 +244,8 @@ gst_faac_init (GstFaac * faac)
   faac->midside = TRUE;
 }
 
-static GstPadLinkReturn
-gst_faac_sinkconnect (GstPad * pad, const GstCaps * caps)
+static gboolean
+gst_faac_sink_setcaps (GstPad * pad, GstCaps * caps)
 {
   GstFaac *faac = GST_FAAC (gst_pad_get_parent (pad));
   GstStructure *structure = gst_caps_get_structure (caps, 0);
@@ -258,7 +254,7 @@ gst_faac_sinkconnect (GstPad * pad, const GstCaps * caps)
   gulong samples, bytes, fmt = 0, bps = 0;
 
   if (!gst_caps_is_fixed (caps))
-    return GST_PAD_LINK_DELAYED;
+    return FALSE;               /* GST_PAD_LINK_DELAYED; */
 
   if (faac->handle) {
     faacEncClose (faac->handle);
@@ -275,7 +271,7 @@ gst_faac_sinkconnect (GstPad * pad, const GstCaps * caps)
 
   /* open a new handle to the encoder */
   if (!(handle = faacEncOpen (samplerate, channels, &samples, &bytes)))
-    return GST_PAD_LINK_REFUSED;
+    return FALSE;
 
   switch (depth) {
     case 16:
@@ -294,7 +290,7 @@ gst_faac_sinkconnect (GstPad * pad, const GstCaps * caps)
 
   if (!fmt) {
     faacEncClose (handle);
-    return GST_PAD_LINK_REFUSED;
+    return FALSE;
   }
 
   faac->format = fmt;
@@ -307,21 +303,21 @@ gst_faac_sinkconnect (GstPad * pad, const GstCaps * caps)
 
   /* if the other side was already set-up, redo that */
   if (GST_PAD_CAPS (faac->srcpad))
-    return gst_faac_srcconnect (faac->srcpad,
+    return gst_faac_src_setcaps (faac->srcpad,
         gst_pad_get_allowed_caps (faac->srcpad));
 
   /* else, that'll be done later */
-  return GST_PAD_LINK_OK;
+  return TRUE;
 }
 
-static GstPadLinkReturn
-gst_faac_srcconnect (GstPad * pad, const GstCaps * caps)
+static gboolean
+gst_faac_src_setcaps (GstPad * pad, GstCaps * caps)
 {
   GstFaac *faac = GST_FAAC (gst_pad_get_parent (pad));
   gint n;
 
   if (!faac->handle || (faac->samplerate == -1 || faac->channels == -1)) {
-    return GST_PAD_LINK_DELAYED;
+    return FALSE;
   }
 
   /* we do samplerate/channels ourselves */
@@ -339,7 +335,6 @@ gst_faac_srcconnect (GstPad * pad, const GstCaps * caps)
     faacEncConfiguration *conf;
     gint mpegversion = 0;
     GstCaps *newcaps;
-    GstPadLinkReturn ret;
 
     gst_structure_get_int (structure, "mpegversion", &mpegversion);
 
@@ -363,28 +358,25 @@ gst_faac_srcconnect (GstPad * pad, const GstCaps * caps)
         "mpegversion", G_TYPE_INT, mpegversion,
         "channels", G_TYPE_INT, faac->channels,
         "rate", G_TYPE_INT, faac->samplerate, NULL);
-    ret = gst_pad_try_set_caps (faac->srcpad, newcaps);
-
-    switch (ret) {
-      case GST_PAD_LINK_OK:
-      case GST_PAD_LINK_DONE:
-        return GST_PAD_LINK_DONE;
-      case GST_PAD_LINK_DELAYED:
-        return GST_PAD_LINK_DELAYED;
-      default:
-        break;
-    }
+
+    /* negotiate with these caps */
+    GST_DEBUG ("here are the caps: %" GST_PTR_FORMAT, newcaps);
+    if (gst_pad_set_caps (faac->srcpad, newcaps) == TRUE)
+      return TRUE;
+    else
+      return FALSE;
   }
 
-  return GST_PAD_LINK_REFUSED;
+  return FALSE;
 }
 
-static void
-gst_faac_chain (GstPad * pad, GstData * data)
+static GstFlowReturn
+gst_faac_chain (GstPad * pad, GstBuffer * data)
 {
   GstFaac *faac = GST_FAAC (gst_pad_get_parent (pad));
   GstBuffer *inbuf, *outbuf, *subbuf;
   guint size, ret_size, in_size, frame_size;
+  GstFlowReturn result = GST_FLOW_OK;
 
   if (GST_IS_EVENT (data)) {
     GstEvent *event = GST_EVENT (data);
@@ -399,25 +391,25 @@ gst_faac_chain (GstPad * pad, GstData * data)
             GST_ELEMENT_ERROR (faac, LIBRARY, ENCODE, (NULL), (NULL));
             gst_event_unref (event);
             gst_buffer_unref (outbuf);
-            return;
+            return GST_FLOW_ERROR;
           }
 
           if (ret_size > 0) {
             GST_BUFFER_SIZE (outbuf) = ret_size;
             GST_BUFFER_TIMESTAMP (outbuf) = 0;
             GST_BUFFER_DURATION (outbuf) = 0;
-            gst_pad_push (faac->srcpad, GST_DATA (outbuf));
+            gst_pad_push (faac->srcpad, outbuf);
           } else {
             break;
           }
         }
 
-        gst_element_set_eos (GST_ELEMENT (faac));
+        gst_pad_push_event (faac->srcpad, gst_event_new_eos ());
         gst_pad_push (faac->srcpad, data);
-        return;
+        return result;
       default:
         gst_pad_event_default (pad, event);
-        return;
+        return result;
     }
   }
 
@@ -427,16 +419,16 @@ gst_faac_chain (GstPad * pad, GstData * data)
     GST_ELEMENT_ERROR (faac, CORE, NEGOTIATION, (NULL),
         ("format wasn't negotiated before chain function"));
     gst_buffer_unref (inbuf);
-    return;
+    return GST_FLOW_ERROR;
   }
 
   if (!GST_PAD_CAPS (faac->srcpad)) {
-    if (gst_faac_srcconnect (faac->srcpad,
+    if (gst_faac_src_setcaps (faac->srcpad,
             gst_pad_get_allowed_caps (faac->srcpad)) <= 0) {
       GST_ELEMENT_ERROR (faac, CORE, NEGOTIATION, (NULL),
           ("failed to negotiate MPEG/AAC format with next element"));
       gst_buffer_unref (inbuf);
-      return;
+      return GST_FLOW_ERROR;
     }
   }
 
@@ -473,7 +465,7 @@ gst_faac_chain (GstPad * pad, GstData * data)
         gst_buffer_unref (inbuf);
       }
 
-      return;
+      return result;
     }
 
     /* create the frame */
@@ -506,7 +498,7 @@ gst_faac_chain (GstPad * pad, GstData * data)
       GST_ELEMENT_ERROR (faac, LIBRARY, ENCODE, (NULL), (NULL));
       gst_buffer_unref (inbuf);
       gst_buffer_unref (subbuf);
-      return;
+      return GST_FLOW_ERROR;
     }
 
     if (ret_size > 0) {
@@ -521,7 +513,7 @@ gst_faac_chain (GstPad * pad, GstData * data)
         GST_BUFFER_DURATION (outbuf) += faac->cache_duration;
         faac->cache_duration = 0;
       }
-      gst_pad_push (faac->srcpad, GST_DATA (outbuf));
+      gst_pad_push (faac->srcpad, outbuf);
     } else {
       /* FIXME: what I'm doing here isn't fully correct, but there
        * really isn't a better way yet.
@@ -540,6 +532,8 @@ gst_faac_chain (GstPad * pad, GstData * data)
     in_size -= frame_size;
     gst_buffer_unref (subbuf);
   }
+
+  return result;
 }
 
 static void