Remove module_t.b_builtin

diff --git a/src/modules/entry.c b/src/modules/entry.c
index a6c38cebd3a0ef8cba5e6e120091c6c05ff75ba5..ba5aa75fb3eec4f1f4ce04b8c3bfb79c2bc23e6f 100644 (file)
--- a/src/modules/entry.c
+++ b/src/modules/entry.c
@@ -67,7 +67,6 @@ module_t *vlc_module_create (void)
     /*module->handle = garbage */
     module->psz_filename = NULL;
     module->domain = NULL;
-    module->b_builtin = false;
     module->b_loaded = false;
     return module;
 }
@@ -108,7 +107,6 @@ module_t *vlc_submodule_create (module_t *module)
     submodule->psz_help = NULL;
     submodule->psz_capability = NULL;
     submodule->i_score = module->i_score;
-    submodule->b_builtin = false;
     submodule->b_loaded = false;
     submodule->b_unloadable = false;
     submodule->pf_activate = NULL;

diff --git a/src/modules/modules.c b/src/modules/modules.c
index 915b5d7266dd73e9e9c325992261b2054b77fd2c..e654dd884d1fe9a579b3ac48c9f09f2d18d5069b 100644 (file)
--- a/src/modules/modules.c
+++ b/src/modules/modules.c
@@ -509,8 +509,10 @@ found_shortcut:
         /* Make sure the module is loaded in mem */
         module_t *p_real = p_cand->parent ? p_cand->parent : p_cand;
 
-        if( !p_real->b_builtin && !p_real->b_loaded )
+        if (!p_real->b_loaded)
         {
+            assert (p_real->psz_filename != NULL);
+
             module_t *p_new_module =
                 AllocatePlugin( p_this, p_real->psz_filename, false );
             if( p_new_module == NULL )
@@ -924,7 +926,6 @@ static int AllocatePluginFile( vlc_object_t * p_this, module_bank_t *p_bank,
     assert( p_module->next == NULL );
 
     /* Unload plugin until we really need it */
-    assert( !p_module->b_builtin );
     if( p_module->b_loaded && p_module->b_unloadable )
     {
         module_Unload( p_module->handle );
@@ -981,7 +982,9 @@ static module_t *AllocatePlugin( vlc_object_t * p_this, const char *psz_file,
         return NULL;
     }
 
-    p_module->psz_filename = strdup( psz_file );
+    p_module->psz_filename = strdup (psz_file);
+    if (unlikely(p_module->psz_filename == NULL))
+        goto error;
     p_module->handle = handle;
     p_module->b_loaded = true;
 
@@ -1006,8 +1009,6 @@ static module_t *AllocatePlugin( vlc_object_t * p_this, const char *psz_file,
         msg_Err( p_this, "cannot initialize plugin `%s'", psz_file );
         goto error;
     }
-
-    assert( !p_module->b_builtin );
     return p_module;
 error:
     free( p_module->psz_filename );
@@ -1030,7 +1031,6 @@ static module_t *module_InitStatic (vlc_plugin_cb entry)
     if (entry (module))
         assert (0);
 
-    module->b_builtin = true;
     module->b_loaded = true;
     module->b_unloadable = false;
 
@@ -1060,14 +1060,9 @@ static void DeleteModule (module_t **head, module_t *p_module)
 
     /* We free the structures that we strdup()ed in Allocate*Module(). */
 #ifdef HAVE_DYNAMIC_PLUGINS
-    if( !p_module->b_builtin )
-    {
-        if( p_module->b_loaded && p_module->b_unloadable )
-        {
-            module_Unload( p_module->handle );
-        }
-        free( p_module->psz_filename );
-    }
+    if (p_module->b_loaded && p_module->b_unloadable)
+        module_Unload (p_module->handle);
+    free (p_module->psz_filename);
 #endif
 
     /* Free and detach the object's children */

diff --git a/src/modules/modules.h b/src/modules/modules.h
index f1ba75156b87cce16e49391abe77a7755935906e..e74d5cec04f641298dab1226b30fa6752564dd23 100644 (file)
--- a/src/modules/modules.h
+++ b/src/modules/modules.h
@@ -83,7 +83,6 @@ struct module_t
     char    *psz_capability;                                 /**< Capability */
     int      i_score;                          /**< Score for the capability */
 
-    bool          b_builtin;  /* Set to true if the module is built in */
     bool          b_loaded;        /* Set to true if the dll is loaded */
     bool b_unloadable;                        /**< Can we be dlclosed? */