Hi,
In _ecore_main_select func in ecore_main.c, return value is checked twice by
the different 'if' statement.
I think it would be better to use 'else if {}'.
Would you please review the attached patch?
Index: ecore_main.c
===================================================================
--- ecore_main.c (revision 57581)
+++ ecore_main.c (working copy)
@@ -1149,7 +1149,7 @@ _ecore_main_select(double timeout)
else if (errno == EBADF) _ecore_main_fd_handlers_bads_rem();
#endif
}
- if (ret > 0)
+ else if (ret > 0)
{
#ifdef HAVE_EPOLL
_ecore_main_fdh_poll_mark_active();
------------------------------------------------------------------------------
Colocation vs. Managed Hosting
A question and answer guide to determining the best fit
for your organization - today and in the future.
http://p.sf.net/sfu/internap-sfd2d
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
