cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=b9d8426eb7e6067b9c19cb823ec91f0513dc82eb

commit b9d8426eb7e6067b9c19cb823ec91f0513dc82eb
Author: Srivardhan Hebbar <sri.heb...@samsung.com>
Date:   Fri Oct 30 06:53:36 2015 +0100

    evas: removing redundant check in gl_x11 backend.
    
    Summary:
    Checking for NULL is redundant here, because if cfgs was NULL, then at
    line 760 it would fail.
    
    Signed-off-by: Srivardhan Hebbar <sri.heb...@samsung.com>
    
    Reviewers: cedric
    
    Differential Revision: https://phab.enlightenment.org/D3238
    
    Signed-off-by: Cedric BAIL <ced...@osg.samsung.com>
---
 src/modules/evas/engines/gl_x11/evas_engine.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/modules/evas/engines/gl_x11/evas_engine.c 
b/src/modules/evas/engines/gl_x11/evas_engine.c
index 930092d..59195e8 100644
--- a/src/modules/evas/engines/gl_x11/evas_engine.c
+++ b/src/modules/evas/engines/gl_x11/evas_engine.c
@@ -773,7 +773,7 @@ evgl_eng_pbuffer_surface_create(void *data, EVGL_Surface 
*sfc,
    surface_attrs[i++] = sfc->h;
    surface_attrs[i++] = 0;
    pbuf = glXCreatePbuffer(re->software.ob->disp, cfgs[0], surface_attrs);
-   if (cfgs) XFree(cfgs);
+   XFree(cfgs);
 
    if (!pbuf)
      {

-- 


Reply via email to