[vlc-devel] commit: Don't handle impossible error ( Rémi Denis-Courmont )

git version control git at videolan.org
Tue Feb 24 18:51:50 CET 2009


vlc | branch: master | Rémi Denis-Courmont <rdenis at simphalempin.com> | Tue Feb 24 19:06:37 2009 +0200| [ef48f9b6ff12a989e7b90573d2261a3be128e6a3] | committer: Rémi Denis-Courmont 

Don't handle impossible error

glXChooseFBConfig always return NULL if there are no adequate configs.

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=ef48f9b6ff12a989e7b90573d2261a3be128e6a3
---

 modules/video_output/x11/glx.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/modules/video_output/x11/glx.c b/modules/video_output/x11/glx.c
index b2553ca..919fa40 100644
--- a/modules/video_output/x11/glx.c
+++ b/modules/video_output/x11/glx.c
@@ -243,10 +243,9 @@ int InitGLX13( vout_thread_t *p_vout )
 
     /* Get the FB configuration */
     p_fbconfs = glXChooseFBConfig( p_sys->p_display, 0, p_attr, &i_nbelem );
-    if( !p_fbconfs || (i_nbelem <= 0) )
+    if( p_fbconfs == NULL )
     {
         msg_Err( p_vout, "Cannot get FB configurations");
-        if( p_fbconfs ) XFree( p_fbconfs );
         return VLC_EGENERIC;
     }
     fbconf = p_fbconfs[0];




More information about the vlc-devel mailing list