]> git.sesse.net Git - vlc/blobdiff - modules/video_output/x11/xcommon.c
Removes trailing spaces. Removes tabs.
[vlc] / modules / video_output / x11 / xcommon.c
index 03c8630505145a55b8e661ea1eb28a37a73c889a..20567c3eacf04e7d8ae0c4dd9f076a5239f2f572 100644 (file)
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
-#include <errno.h>                                                 /* ENOMEM */
-#include <stdlib.h>                                                /* free() */
-#include <string.h>                                            /* strerror() */
-
 #include <vlc/vlc.h>
 #include <vlc_interface.h>
 #include <vlc_playlist.h>
 #include <vlc_vout.h>
 #include <vlc_keys.h>
 
+#include <errno.h>                                                 /* ENOMEM */
+
 #ifdef HAVE_MACHINE_PARAM_H
     /* BSD */
 #   include <machine/param.h>
@@ -290,6 +288,47 @@ int E_(Activate) ( vlc_object_t *p_this )
         }
     }
     p_vout->output.i_chroma = X112VLC_FOURCC(p_vout->output.i_chroma);
+#elif defined(MODULE_NAME_IS_glx)
+    {
+        int i_opcode, i_evt, i_err = 0;
+        int i_maj, i_min = 0;
+
+        /* Check for GLX extension */
+        if( !XQueryExtension( p_vout->p_sys->p_display, "GLX",
+                              &i_opcode, &i_evt, &i_err ) )
+        {
+            msg_Err( p_this, "GLX extension not supported" );
+            XCloseDisplay( p_vout->p_sys->p_display );
+            free( p_vout->p_sys );
+            return VLC_EGENERIC;
+        }
+        if( !glXQueryExtension( p_vout->p_sys->p_display, &i_err, &i_evt ) )
+        {
+            msg_Err( p_this, "glXQueryExtension failed" );
+            XCloseDisplay( p_vout->p_sys->p_display );
+            free( p_vout->p_sys );
+            return VLC_EGENERIC;
+        }
+
+        /* Check GLX version */
+        if (!glXQueryVersion( p_vout->p_sys->p_display, &i_maj, &i_min ) )
+        {
+            msg_Err( p_this, "glXQueryVersion failed" );
+            XCloseDisplay( p_vout->p_sys->p_display );
+            free( p_vout->p_sys );
+            return VLC_EGENERIC;
+        }
+        if( i_maj <= 0 || ((i_maj == 1) && (i_min < 3)) )
+        {
+            p_vout->p_sys->b_glx13 = VLC_FALSE;
+            msg_Dbg( p_this, "using GLX 1.2 API" );
+        }
+        else
+        {
+            p_vout->p_sys->b_glx13 = VLC_TRUE;
+            msg_Dbg( p_this, "using GLX 1.3 API" );
+        }
+    }
 #endif
 
     /* Create blank cursor (for mouse cursor autohiding) */
@@ -389,9 +428,9 @@ int E_(Activate) ( vlc_object_t *p_this )
 #ifdef HAVE_XSP
     p_vout->p_sys->i_hw_scale = 1;
 #endif
-    
+
 #ifdef HAVE_OSSO
-    p_vout->p_sys->i_backlight_on_counter = i_backlight_on_interval; 
+    p_vout->p_sys->i_backlight_on_counter = i_backlight_on_interval;
     p_vout->p_sys->p_octx = osso_initialize( "vlc", VERSION, 0, NULL );
     if ( p_vout->p_sys->p_octx == NULL ) {
         msg_Err( p_vout, "Could not get osso context" );
@@ -399,7 +438,7 @@ int E_(Activate) ( vlc_object_t *p_this )
         msg_Dbg( p_vout, "Initialized osso context" );
     }
 #endif
-                   
+
     /* Variable to indicate if the window should be on top of others */
     /* Trigger a callback right now */
     var_Get( p_vout, "video-on-top", &val );
@@ -460,7 +499,7 @@ void E_(Deactivate) ( vlc_object_t *p_this )
 #ifdef HAVE_XSP
     DisablePixelDoubling(p_vout);
 #endif
-           
+    
     DestroyCursor( p_vout );
     EnableXScreenSaver( p_vout );
     DestroyWindow( p_vout, &p_vout->p_sys->original_window );
@@ -478,7 +517,7 @@ void E_(Deactivate) ( vlc_object_t *p_this )
         osso_deinitialize( p_vout->p_sys->p_octx );
     }
 #endif
-       
+
     free( p_vout->p_sys );
 }
 
@@ -995,7 +1034,7 @@ static void DisplayVideo( vout_thread_t *p_vout, picture_t *p_pic )
                     first_field);
 
     XVMCUNLOCKDISPLAY( p_vout->p_sys->p_display );
-    if( p_vout->p_sys->xvmc_deinterlace_method == 2 ) 
+    if( p_vout->p_sys->xvmc_deinterlace_method == 2 )
     {   /* BOB DEINTERLACE */
         if( p_picture->p_sys->nb_display == 0 )/* && ((t2-t1) < 15000)) */
         {
@@ -1406,41 +1445,17 @@ static int ManageVideo( vout_thread_t *p_vout )
      */
     if ( p_vout->i_changes & VOUT_FULLSCREEN_CHANGE )
     {
-        vlc_value_t val;
+        vlc_value_t val_fs, val_ontop;
 
         /* Update the object variable and trigger callback */
-        val.b_bool = !p_vout->b_fullscreen;
+        val_fs.b_bool = !p_vout->b_fullscreen;
 
-        /*
-         * FIXME FIXME FIXME FIXME: EXPLICIT HACK.
-         * On the one hand, we cannot hold the lock while triggering a
-         * callback, as it causes a deadlock with video-on-top handling.
-         * On the other hand, we have to lock while triggering the
-         * callback to:
-         *  1/ make sure video-on-top remains in sync with fullscreen
-         *    (i.e. unlocking creates a race condition if fullscreen is
-         *     switched on and off VERY FAST).
-         *  2/ avoid possible corruption bugs if another thread gets the
-         *     mutex and modifies our data in-between.
-         *
-         * This is obviously contradictory. Correct solutions may include:
-         *  - putting the fullscreen NAND video-on-top logic out of libvlc,
-         *    back into the video output plugins (ugly code duplication...),
-         *  - serializing fullscreen and video-on-top handling properly
-         *    instead of doing it via the fullscreen callback. That's got to
-         *    be the correct one.
-         */
-#ifdef MODULE_NAME_IS_xvmc
-        xvmc_context_reader_unlock( &p_vout->p_sys->xvmc_lock );
-#endif
-        vlc_mutex_unlock( &p_vout->p_sys->lock );
-
-        var_Set( p_vout, "fullscreen", val );
+        var_Set( p_vout, "fullscreen", val_fs );
 
-        vlc_mutex_lock( &p_vout->p_sys->lock );
-#ifdef MODULE_NAME_IS_xvmc
-        xvmc_context_reader_lock( &p_vout->p_sys->xvmc_lock );
-#endif
+        /* Disable "always on top" in fullscreen mode */
+        var_Get( p_vout, "video-on-top", &val_ontop );
+        if( val_ontop.b_bool )
+            WindowOnTop( p_vout, val_fs.b_bool );
 
         ToggleFullScreen( p_vout );
         p_vout->i_changes &= ~VOUT_FULLSCREEN_CHANGE;
@@ -1504,13 +1519,13 @@ static int ManageVideo( vout_thread_t *p_vout )
 #ifdef MODULE_NAME_IS_xvmc
     xvmc_context_reader_unlock( &p_vout->p_sys->xvmc_lock );
 #endif
-    
 #ifdef HAVE_OSSO
     if ( p_vout->p_sys->p_octx != NULL ) {
         if ( p_vout->p_sys->i_backlight_on_counter == i_backlight_on_interval ) {
             if ( osso_display_blanking_pause( p_vout->p_sys->p_octx ) != OSSO_OK ) {
                 msg_Err( p_vout, "Could not disable backlight blanking" );
-           } else {
+        } else {
                 msg_Dbg( p_vout, "Backlight blanking disabled" );
             }
             p_vout->p_sys->i_backlight_on_counter = 0;
@@ -1519,7 +1534,7 @@ static int ManageVideo( vout_thread_t *p_vout )
         }
     }
 #endif
-           
+    
     vlc_mutex_unlock( &p_vout->p_sys->lock );
     return 0;
 }
@@ -1619,6 +1634,26 @@ static int CreateWindow( vout_thread_t *p_vout, x11_window_t *p_win )
                            CWBackingStore | CWBackPixel | CWEventMask,
                            &xwindow_attributes );
 
+        var_Get( p_vout, "video-title", &val );
+        if( !val.psz_string || !*val.psz_string )
+        {
+            XStoreName( p_vout->p_sys->p_display, p_win->base_window,
+#ifdef MODULE_NAME_IS_x11
+                        VOUT_TITLE " (X11 output)"
+#elif defined(MODULE_NAME_IS_glx)
+                        VOUT_TITLE " (GLX output)"
+#else
+                        VOUT_TITLE " (XVideo output)"
+#endif
+              );
+        }
+        else
+        {
+            XStoreName( p_vout->p_sys->p_display,
+                        p_win->base_window, val.psz_string );
+        }
+        if( val.psz_string ) free( val.psz_string );
+
         if( !p_vout->b_fullscreen )
         {
             /* Set window manager hints and properties: size hints, command,
@@ -1645,28 +1680,6 @@ static int CreateWindow( vout_thread_t *p_vout, x11_window_t *p_win )
                                  (unsigned char *)&mwmhints,
                                  PROP_MWM_HINTS_ELEMENTS );
             }
-            else
-            {
-                 var_Get( p_vout, "video-title", &val );
-                 if( !val.psz_string || !*val.psz_string )
-                 {
-                    XStoreName( p_vout->p_sys->p_display, p_win->base_window,
-#ifdef MODULE_NAME_IS_x11
-                                VOUT_TITLE " (X11 output)"
-#elif defined(MODULE_NAME_IS_glx)
-                                VOUT_TITLE " (GLX output)"
-#else
-                                VOUT_TITLE " (XVideo output)"
-#endif
-                      );
-                }
-                else
-                {
-                    XStoreName( p_vout->p_sys->p_display,
-                               p_win->base_window, val.psz_string );
-                }
-                if( val.psz_string ) free( val.psz_string );
-            }
         }
     }
     else
@@ -1895,7 +1908,7 @@ static int NewPicture( vout_thread_t *p_vout, picture_t *p_pic )
         p_pic->p_sys->p_image =
             CreateImage( p_vout, p_vout->p_sys->p_display,
 #if defined(MODULE_NAME_IS_xvideo) || defined(MODULE_NAME_IS_xvmc)
-                         p_vout->p_sys->i_xvport, 
+                         p_vout->p_sys->i_xvport,
                          VLC2X11_FOURCC(p_vout->output.i_chroma),
                          p_pic->format.i_bits_per_pixel,
 #else
@@ -2219,7 +2232,7 @@ static void ToggleFullScreen ( vout_thread_t *p_vout )
 #ifdef HAVE_XSP
         EnablePixelDoubling( p_vout );
 #endif
-               
+        
     }
     else
     {
@@ -2897,7 +2910,7 @@ static int X11ErrorHandler( Display * display, XErrorEvent * event )
     switch( event->request_code )
     {
     case X_SetInputFocus:
-        /* Ingnore errors on XSetInputFocus()
+        /* Ignore errors on XSetInputFocus()
          * (they happen when a window is not yet mapped) */
         return 0;