Changeset 187a6ea2ec52ec3624bd8478c7fc0c6a46fb43bf

Timestamp:

09/17/06 12:17:39 (2 years ago)

Author:

Clément Stenac <zorglub@videolan.org>

git-committer:

Clément Stenac <zorglub@videolan.org> 1158488259 +0000

git-parent:

[c17e7670f4f4e40e5da47f463852f1af833cf662]

git-author:

Clément Stenac <zorglub@videolan.org> 1158488259 +0000

Message:

Interfaces are now allowed not to have a Run function.
This will help reduce useless process wakeups

Files:

• modules/misc/notify/notify.c (modified) (4 diffs)
• src/interface/interface.c (modified) (3 diffs)

modules/misc/notify/notify.c

@@ -39 +39 @@
-static int  Open    ( vlc_object_t * );
-static void Close   ( vlc_object_t * );
-static void Run     ( intf_thread_t * );
-
-static int ItemChange( vlc_object_t *, const char *,
@@ -49 +48 @@
- * Module descriptor
- ****************************************************************************/
-
-
-#define APPLICATION_NAME "VLC media player"
@@ -87 +85 @@
-    pl_Release( p_intf );
-
-    p_intf->pf_run = Run;
-    return VLC_SUCCESS;
-}
@@ -101 +98 @@
-
-    notify_uninit();
-}
-
-/*****************************************************************************
- * Run
- *****************************************************************************/
-static void Run( intf_thread_t *p_this )
-{
-    msleep( 10*INTF_IDLE_SLEEP );
-}
-

src/interface/interface.c

@@ -187 +187 @@
-    else
-    {
+        /* This interface doesn't need to be run */
+        if( !p_intf->pf_run )
+        {
+            return VLC_SUCCESS;
+        }
-        /* Run the interface in a separate thread */
-        if( !strcmp( p_intf->p_module->psz_object_name, "macosx" ) )
@@ -227 +232 @@
-    else
-    {
+        /* This interface doesn't need to be run */
+        if( !p_intf->pf_run )
+            return VLC_SUCCESS;
+
-        /* Run the interface in a separate thread */
-        if( vlc_thread_create( p_intf, "interface", RunInterface,
@@ -256 +265 @@
-
-    /* Wait for the thread to exit */
-
+
+
-}
-