plugin: warn if plugin name starts with a "
authorTim-Philipp Müller <tim@centricular.net>
Thu, 2 Aug 2012 23:05:53 +0000 (00:05 +0100)
committerTim-Philipp Müller <tim@centricular.net>
Thu, 2 Aug 2012 23:05:53 +0000 (00:05 +0100)
This can easily happen as side-effect of the plugin name
in GST_PLUGIN_DEFINE no longer being a string in 0.11, but
a name to G_STRINGIFY.

gst/gstplugin.c

index 9c8df76..0ee39a3 100644 (file)
@@ -786,6 +786,11 @@ gst_plugin_load_file (const gchar * filename, GError ** error)
     CHECK_PLUGIN_DESC_FIELD (plugin->orig_desc, package, filename);
     CHECK_PLUGIN_DESC_FIELD (plugin->orig_desc, origin, filename);
 
+    if (plugin->orig_desc->name != NULL && plugin->orig_desc->name[0] == '"') {
+      g_warning ("Invalid plugin name '%s' - fix your GST_PLUGIN_DEFINE "
+          "(remove quotes around plugin name)", plugin->orig_desc->name);
+    }
+
     if (plugin->orig_desc->release_datetime != NULL &&
         !check_release_datetime (plugin->orig_desc->release_datetime)) {
       GST_ERROR ("GstPluginDesc for '%s' has invalid datetime '%s'",