From: Vikram Narayanan <vikram186@gmail.com>

Subject: [E-devel] [PATCH] ecore/ecore_fb: Tweaks in _ecore_fb_size_get

As the width and height are defined as static, no need to initialize it 
again to zero. Also removed checks for 'valid pointers' as it is redundant.



SVN revision: 69556
This commit is contained in:
Vikram Narayanan 2012-03-22 06:01:59 +00:00 committed by Carsten Haitzler
parent f33084555f
commit 77a76443ec
2 changed files with 11 additions and 13 deletions

View File

@ -51,3 +51,4 @@ Bluezery <ohpowel@gmail.com>
Doyoun Kang <wayofmine@gmail.com> <doyoun.kang@samsung.com> Doyoun Kang <wayofmine@gmail.com> <doyoun.kang@samsung.com>
Haifeng Deng <haifeng.deng@samsung.com> Haifeng Deng <haifeng.deng@samsung.com>
Jérémy Zurcher <jeremy@asynk.ch> Jérémy Zurcher <jeremy@asynk.ch>
Vikram Narayanan <vikram186@gmail.com>

View File

@ -92,21 +92,18 @@ _ecore_fb_size_get(int *w, int *h)
fb = open("/dev/fb0", O_RDWR); fb = open("/dev/fb0", O_RDWR);
if (fb < 0) if (fb < 0)
{ goto exit;
if (w) *w = 0;
if (h) *h = 0;
return;
}
if (ioctl(fb, FBIOGET_VSCREENINFO, &fb_var) == -1) if (ioctl(fb, FBIOGET_VSCREENINFO, &fb_var) == -1)
{ goto err_ioctl;
if (w) *w = 0;
if (h) *h = 0; *w = fb_var.xres;
close(fb); *h = fb_var.yres;
return;
} err_ioctl:
close(fb); close(fb);
if (w) *w = fb_var.xres; exit:
if (h) *h = fb_var.yres; return;
} }
/** /**