]> git.sesse.net Git - vlc/blobdiff - modules/video_output/drawable.c
Do not add a newline at the end of the strings send to msg_*
[vlc] / modules / video_output / drawable.c
index f40b8c3b09403d4bb2c6988696f68664adca56e5..b8f7cec1a24bd4872d7811fe66ee9ef42d1230a9 100644 (file)
@@ -7,7 +7,7 @@
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU General Public License
- * as published by the Free Software Foundation; either version 2.0
+ * as published by the Free Software Foundation; either version 2
  * of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ -15,7 +15,7 @@
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
- * You should have received a copy of the GNU Lesser General Public
+ * You should have received a copy of the GNU General Public
  * License along with this library; if not, write to the Free Software
  * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
  ****************************************************************************/
 #include <vlc_plugin.h>
 #include <vlc_vout_window.h>
 
-static int  Open (vlc_object_t *);
-static void Close(vlc_object_t *);
+#define HWND_TEXT N_("Window handle (HWND)")
+#define HWND_LONGTEXT N_( \
+    "Video will be embedded in this pre-existing window. " \
+    "If zero, a new window will be created.")
+
+static int  Open (vout_window_t *, const vout_window_cfg_t *);
+static void Close(vout_window_t *);
 
 /*
  * Module descriptor
@@ -42,91 +47,76 @@ vlc_module_begin ()
     set_description (N_("Embedded window video"))
     set_category (CAT_VIDEO)
     set_subcategory (SUBCAT_VIDEO_VOUT)
-    set_capability ("vout window xid", 70)
+    set_capability ("vout window hwnd", 0)
     set_callbacks (Open, Close)
-    //add_integer ("drawable-hwnd", 0, NULL, HWN_TEXT, HWND_LONGTEXT, true) /* How to ? */
-    //    change_unsaveable ()
+    add_shortcut ("embed-hwnd")
+
+    add_integer ("drawable-hwnd", 0, HWND_TEXT, HWND_LONGTEXT, true)
+        change_volatile ()
 vlc_module_end ()
 
 static int Control (vout_window_t *, int, va_list);
 
-/* TODO: move to vlc_variables.h */
-static inline void *var_GetAddress (vlc_object_t *o, const char *name)
-{
-    vlc_value_t val;
-    return var_Get (o, name, &val) ? NULL : val.p_address;
-}
-
+/* Keep a list of busy drawables, so we don't overlap videos if there are
+ * more than one video track in the stream. */
 static vlc_mutex_t serializer = VLC_STATIC_MUTEX;
+static uintptr_t *used = NULL;
 
 /**
  * Find the drawable set by libvlc application.
  */
-static int Open (vlc_object_t *obj)
+static int Open (vout_window_t *wnd, const vout_window_cfg_t *cfg)
 {
-    vout_window_t *wnd = (vout_window_t *)obj;
-    void **used, *val;
-    size_t n = 0;
-
-    if (var_Create (obj->p_libvlc, "hwnd-in-use", VLC_VAR_ADDRESS)
-     || var_Create (obj, "drawable-hwnd", VLC_VAR_DOINHERIT | VLC_VAR_ADDRESS))
-        return VLC_ENOMEM;
+    VLC_UNUSED (cfg);
+    uintptr_t val = var_InheritInteger (wnd, "drawable-hwnd");
+    if (val == 0)
+        return VLC_EGENERIC;
 
-    val = var_GetAddress (obj, "drawable-hwnd");
-    var_Destroy (obj, "drawable-hwn");
+    uintptr_t *tab;
+    size_t n = 0;
 
-    /* Keep a list of busy drawables, so we don't overlap videos if there are
-     * more than one video track in the stream. */
     vlc_mutex_lock (&serializer);
-    /* TODO: per-type list of busy drawables */
-    used = var_GetAddress (VLC_OBJECT (obj->p_libvlc), "drawables-in-use");
     if (used != NULL)
-    {
-        while (used[n] != NULL)
-        {
+        for (/*n = 0*/; used[n]; n++)
             if (used[n] == val)
+            {
+                msg_Warn (wnd, "HWND 0x%zX is busy", val);
+                val = 0;
                 goto skip;
-            n++;
-        }
-    }
+            }
 
-    used = realloc (used, sizeof (*used) * (n + 2));
-    if (used != NULL)
+    tab = realloc (used, sizeof (*used) * (n + 2));
+    if (likely(tab != NULL))
     {
+        used = tab;
         used[n] = val;
-        used[n + 1] = NULL;
-        var_SetAddress (obj->p_libvlc, "hwnd-in-use", used);
+        used[n + 1] = 0;
     }
     else
-    {
+        val = 0;
 skip:
-        msg_Warn (wnd, "HWND %p is busy", val);
-        val = NULL;
-    }
     vlc_mutex_unlock (&serializer);
 
-    if (val == NULL)
+    if (val == 0)
         return VLC_EGENERIC;
 
-    wnd->handle.hwnd = val;
+    wnd->handle.hwnd = (void *)val;
     wnd->control = Control;
-    wnd->sys = val;
+    wnd->sys = (void *)val;
     return VLC_SUCCESS;
 }
 
 /**
  * Release the drawable.
  */
-static void Close (vlc_object_t *obj)
+static void Close (vout_window_t *wnd)
 {
-    vout_window_t *wnd = (vout_window_t *)obj;
-    void **used, *val = wnd->sys;
+    uintptr_t val = (uintptr_t)wnd->sys;
     size_t n = 0;
 
     /* Remove this drawable from the list of busy ones */
     vlc_mutex_lock (&serializer);
-    used = var_GetAddress (VLC_OBJECT (obj->p_libvlc), "hwnd-in-use");
-    assert (used);
+    assert (used != NULL);
     while (used[n] != val)
     {
         assert (used[n]);
@@ -134,29 +124,28 @@ static void Close (vlc_object_t *obj)
     }
     do
         used[n] = used[n + 1];
-    while (used[++n] != NULL);
+    while (used[++n] != 0);
 
     if (n == 0)
-         var_SetAddress (obj->p_libvlc, "hwnd-in-use", NULL);
+    {
+         free (used);
+         used = NULL;
+    }
     vlc_mutex_unlock (&serializer);
-
-    if (n == 0)
-        free (used);
-    /* Variables are reference-counted... */
-    var_Destroy (obj->p_libvlc, "hwnd-in-use");
 }
 
 
 static int Control (vout_window_t *wnd, int query, va_list ap)
 {
+    VLC_UNUSED( ap );
+
     switch (query)
     {
         case VOUT_WINDOW_SET_SIZE:   /* not allowed */
-        case VOUT_WINDOW_SET_ON_TOP: /* not allowed either, would be ugly */
+        case VOUT_WINDOW_SET_STATE: /* not allowed either, would be ugly */
             return VLC_EGENERIC;
         default:
             msg_Warn (wnd, "unsupported control query %d", query);
             return VLC_EGENERIC;
     }
 }
-