qtwrapper: Replace the hackish 42 magic number with a define
authorMartin Storsjo <martin@martin.st>
Tue, 3 May 2011 11:14:20 +0000 (14:14 +0300)
committerSebastian Dröge <sebastian.droege@collabora.co.uk>
Mon, 30 May 2011 06:46:04 +0000 (08:46 +0200)
sys/qtwrapper/audiodecoders.c

index 718d30c..d7791d0 100644 (file)
@@ -62,6 +62,8 @@
 
 #define QTWRAPPER_ADEC_PARAMS_QDATA g_quark_from_static_string("qtwrapper-adec-params")
 
+#define NO_MORE_INPUT_DATA 42
+
 static GstStaticPadTemplate src_templ = GST_STATIC_PAD_TEMPLATE ("src",
     GST_PAD_SRC,
     GST_PAD_ALWAYS,
@@ -745,9 +747,9 @@ process_buffer_cb (ComponentInstance inAudioConverter,
     return noErr;
   }
 
-  GST_LOG_OBJECT (qtwrapper, "No remaining input data, returning 42 for hack");
+  GST_LOG_OBJECT (qtwrapper, "No remaining input data, returning NO_MORE_INPUT_DATA");
 
-  return 42;
+  return NO_MORE_INPUT_DATA;
 }
 
 static GstFlowReturn
@@ -805,8 +807,7 @@ qtwrapper_audio_decoder_chain (GstPad * pad, GstBuffer * buf)
         (SCAudioInputDataProc) process_buffer_cb,
         qtwrapper, (UInt32 *) & outsamples, qtwrapper->bufferlist, NULL);
 
-    /* TODO: What's this '42' crap?? It does seem to be needed, though. */
-    if ((status != noErr) && (status != 42)) {
+    if ((status != noErr) && (status != NO_MORE_INPUT_DATA)) {
       if (status < 0)
         GST_WARNING_OBJECT (qtwrapper,
             "Error in SCAudioFillBuffer() : %d", (gint32) status);