]> git.sesse.net Git - vlc/blobdiff - modules/misc/lua/meta.c
Clone video filter : fix potential memleak.
[vlc] / modules / misc / lua / meta.c
index 704a8cbeca08dfea740262598530da0ea41a1b13..524ce2a71ec8011a5a2c9c79ecaa9a32acc70604 100644 (file)
@@ -1,7 +1,7 @@
 /*****************************************************************************
  * meta.c: Get meta/artwork using lua scripts
  *****************************************************************************
- * Copyright (C) 2007 the VideoLAN team
+ * Copyright (C) 2007-2008 the VideoLAN team
  * $Id$
  *
  * Authors: Antoine Cellerier <dionoea at videolan tod org>
@@ -33,7 +33,7 @@
 # include "config.h"
 #endif
 
-#include <vlc/vlc.h>
+#include <vlc_common.h>
 #include <vlc_input.h>
 #include <vlc_playlist.h>
 #include <vlc_meta.h>
 #include <vlc_charset.h>
 
 #include "vlc.h"
+#include "libs.h"
 
 
 /*****************************************************************************
  * Local prototypes
  *****************************************************************************/
-static int fetch_meta( vlc_object_t *p_this, const char * psz_filename,
-                       lua_State * L, void * user_data );
 static int fetch_art( vlc_object_t *p_this, const char * psz_filename,
                       lua_State * L, void * user_data );
 static lua_State *vlclua_meta_init( vlc_object_t *p_this,
                                     input_item_t * p_item );
 
 
-/*****************************************************************************
- * Lua function bridge
- *****************************************************************************/
-/* Functions to register */
-static luaL_Reg p_reg[] =
-{
-    /* TODO: make an object out of the stream stuff, so we can do something
-     * like:
-     *
-     * s = vlc.stream_new( "http://www.videolan.org" )
-     * page = s:read( 2^16 )
-     * s:delete()
-     *
-     * instead of (which could be problematic since we don't check if
-     * s is really a stream object):
-     *
-     * s = vlc.stream_new( "http://www.videolan.org" )
-     * page = vlc.stream_read( s, 2^16 )
-     * vlc.stream_delete( s )
-     */
-    { "stream_new", vlclua_stream_new },
-    { "stream_read", vlclua_stream_read },
-    { "stream_readline", vlclua_stream_readline },
-    { "stream_delete", vlclua_stream_delete },
-    { "decode_uri", vlclua_decode_uri },
-    { "resolve_xml_special_chars", vlclua_resolve_xml_special_chars },
-    { "msg_dbg", vlclua_msg_dbg },
-    { "msg_warn", vlclua_msg_warn },
-    { "msg_err", vlclua_msg_err },
-    { "msg_info", vlclua_msg_info },
-    { NULL, NULL }
-};
-
 /*****************************************************************************
  * Init lua
  *****************************************************************************/
+static luaL_Reg p_reg[] = { { NULL, NULL } };
+
 static lua_State * vlclua_meta_init( vlc_object_t *p_this, input_item_t * p_item )
 {
     lua_State * L = luaL_newstate();
@@ -107,6 +75,10 @@ static lua_State * vlclua_meta_init( vlc_object_t *p_this, input_item_t * p_item
 
     luaL_register( L, "vlc", p_reg );
 
+    luaopen_msg( L );
+    luaopen_stream( L );
+    luaopen_strings( L );
+
     lua_pushlightuserdata( L, p_this );
     lua_setfield( L, -2, "private" );
 
@@ -211,91 +183,10 @@ static int fetch_art( vlc_object_t *p_this, const char * psz_filename,
     return i_ret;
 }
 
-/*****************************************************************************
- * Called through lua_scripts_batch_execute to call 'fetch_meta' on the script
- * pointed by psz_filename.
- *****************************************************************************/
-static int fetch_meta( vlc_object_t *p_this, const char * psz_filename,
-                       lua_State * L, void * user_data )
-{
-    input_item_t * p_input = user_data;
-
-    /* In lua, setting a variable to nil is equivalent to deleting it */
-    lua_pushnil( L );
-    lua_setglobal( L, "fetch_meta" );
-
-    /* Load and run the script(s) */
-    if( luaL_dofile( L, psz_filename ) )
-    {
-        msg_Warn( p_this, "Error loading script %s: %s", psz_filename,
-                  lua_tostring( L, lua_gettop( L ) ) );
-        lua_pop( L, 1 );
-        return VLC_EGENERIC;
-    }
-
-    lua_getglobal( L, "fetch_meta" );
-
-    if( !lua_isfunction( L, lua_gettop( L ) ) )
-    {
-        msg_Warn( p_this, "Error while runing script %s, "
-                  "function fetch_meta() not found", psz_filename );
-        lua_pop( L, 1 );
-        return VLC_EGENERIC;
-    }
-
-    if( lua_pcall( L, 0, 1, 0 ) )
-    {
-        msg_Warn( p_this, "Error while runing script %s, "
-                  "function fetch_meta(): %s", psz_filename,
-                  lua_tostring( L, lua_gettop( L ) ) );
-        lua_pop( L, 1 );
-        return VLC_EGENERIC;
-    }
-
-
-    if( lua_gettop( L ) )
-    {
-        if( lua_istable( L, -1 ) )
-        {
-            /* ... item */
-            vlclua_read_meta_data( p_this, L, p_input );
-            vlclua_read_custom_meta_data( p_this, L, p_input );
-        }
-        else
-        {
-            msg_Err( p_this, "Lua playlist script %s: "
-                     "didn't return a table", psz_filename );
-        }
-    }
-    else
-    {
-        msg_Err( p_this, "Script went completely foobar" );
-    }
-
-    /* We tell the batch thing to continue, hence all script
-     * will get the change to add its meta. This behaviour could
-     * be changed. */
-    return VLC_EGENERIC;
-}
-
-/*****************************************************************************
- * Module entry point for meta.
- *****************************************************************************/
-int E_(FindMeta)( vlc_object_t *p_this )
-{
-    meta_engine_t *p_me = (meta_engine_t *)p_this;
-    input_item_t *p_item = p_me->p_item;
-    lua_State *L = vlclua_meta_init( p_this, p_item );
-
-    int i_ret = vlclua_scripts_batch_execute( p_this, "meta", &fetch_meta, L, p_item );
-    lua_close( L );
-    return i_ret;
-}
-
 /*****************************************************************************
  * Module entry point for art.
  *****************************************************************************/
-int E_(FindArt)( vlc_object_t *p_this )
+int FindArt( vlc_object_t *p_this )
 {
     playlist_t *p_playlist = (playlist_t *)p_this;
     input_item_t *p_item = (input_item_t *)(p_playlist->p_private);