brl/mutter
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[x11] Do not needlessly check XVisualInfo

Browse Source 
Since we are destroying any previously set VisualInfo we keep we know
for sure that stage->xvisinfo is going to be None; hence, no reason to
check this condition.
This commit is contained in:
Emmanuele Bassi 2009-05-11 12:39:46 +01:00
parent 0acb98e987
commit 9582fddb24
1 changed files with 7 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
clutter/glx/clutter-stage-glx.c
View File

@ -126,25 +126,23 @@ clutter_stage_glx_realize (ClutterActor *actor)
{ {
GLX_RGBA, GLX_RGBA,
GLX_DOUBLEBUFFER, GLX_DOUBLEBUFFER,
GLX_RED_SIZE, 1, GLX_RED_SIZE, 1,
GLX_GREEN_SIZE, 1, GLX_GREEN_SIZE, 1,
GLX_BLUE_SIZE, 1, GLX_BLUE_SIZE, 1,
GLX_STENCIL_SIZE, 1, GLX_STENCIL_SIZE, 1,
0 0
}; };
if (stage_x11->xvisinfo) if (stage_x11->xvisinfo != None)
{ {
XFree (stage_x11->xvisinfo); XFree (stage_x11->xvisinfo);
stage_x11->xvisinfo = None; stage_x11->xvisinfo = None;
} }
/* The following check seems strange */ stage_x11->xvisinfo = glXChooseVisual (stage_x11->xdpy,
stage_x11->xscreen,
gl_attributes);
if (stage_x11->xvisinfo == None) if (stage_x11->xvisinfo == None)
stage_x11->xvisinfo = glXChooseVisual (stage_x11->xdpy,
stage_x11->xscreen,
gl_attributes);
if (!stage_x11->xvisinfo)
{ {
g_critical ("Unable to find suitable GL visual."); g_critical ("Unable to find suitable GL visual.");
goto fail; goto fail;