and small improvement in naming of the FFMPEG encoders. This makes their naming look...
authorRonald S. Bultje <rbultje@ronald.bitfreak.net>
Mon, 20 Oct 2003 08:24:39 +0000 (08:24 +0000)
committerRonald S. Bultje <rbultje@ronald.bitfreak.net>
Mon, 20 Oct 2003 08:24:39 +0000 (08:24 +0000)
Original commit message from CVS:
Typo (> instead of <) in codec selection (we don't want raw audio to raw audio encoders), and small improvement in naming of the FFMPEG encoders. This makes their naming look slightly better in apps using element_factory->details->longname.

ext/ffmpeg/gstffmpegenc.c

index 54fc65e..d93d60b 100644 (file)
@@ -520,7 +520,7 @@ gst_ffmpegenc_register (GstPlugin *plugin)
     /* no quasi codecs, please */
     if (in_plugin->id == CODEC_ID_RAWVIDEO ||
        (in_plugin->id >= CODEC_ID_PCM_S16LE &&
-        in_plugin->id >= CODEC_ID_PCM_ALAW)) {
+        in_plugin->id <= CODEC_ID_PCM_ALAW)) {
       goto next;
     }
 
@@ -550,7 +550,8 @@ gst_ffmpegenc_register (GstPlugin *plugin)
 
     /* construct the element details struct */
     details = g_new0 (GstElementDetails,1);
-    details->longname = g_strdup(in_plugin->name);
+    details->longname = g_strdup_printf("FFMPEG %s encoder",
+                                       in_plugin->name);
     details->klass = g_strdup_printf("Codec/%s/Encoder",
                                     (in_plugin->type == CODEC_TYPE_VIDEO) ?
                                     "Video" : "Audio");