On Mon, 2010-05-17 at 18:18 +0900, Mike McCormack wrote:
> Dejun.Liu wrote:
> > 
> > On Mon, 2010-05-17 at 14:16 +0900, Mike McCormack wrote:
> >> Hey Guys,
> >>
> >> Here's a patch to support epoll in ecore.
> >>
> >> Probably needs a bit more testing, but let me know what you think.
> >>
> > 
> > why trigger method not be considered? sample code only uses level
> > trigger. 
> 
> This code replaces select, which is level triggered.
> Implementing edge triggered epoll would probably require changing ecore's 
> interfaces.


yes,in ecore level triggered is enough.i agree with you.

but hava another Q.

+void _ecore_main_loop_init(void)
+{
+   epoll_fd = epoll_create(1);
+   if (epoll_fd < 0)
+     ERR("Failed to create epoll fd!");
+   INF("epoll_fd = %d", epoll_fd);
+}

i think in the above function epoll_fd created by epoll_create should be
closed ,so a ecore_epoll_shutdown function may be provided?

and  also why epoll_create's parameter is 1?

sorry for not familar with epoll 

Seven




---------------------------------------------------------------------------------------------------
Confidentiality Notice: The information contained in this e-mail and any 
accompanying attachment(s) 
is intended only for the use of the intended recipient and may be confidential 
and/or privileged of 
Neusoft Corporation, its subsidiaries and/or its affiliates. If any reader of 
this communication is 
not the intended recipient, unauthorized use, forwarding, printing,  storing, 
disclosure or copying 
is strictly prohibited, and may be unlawful.If you have received this 
communication in error,please 
immediately notify the sender by return e-mail, and delete the original message 
and all copies from 
your system. Thank you. 
---------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------

_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to