Hi,

Thank you for the patch!
I had a quick look, I don't know anything about the Set/ReleaseCapture, so
I don't know if that's good or not and I can't comment on it.

I noticed though you added a free callback in your code for the event, but
it's not needed since ecore will free the event itself.
See the code here :
https://github.com/kakaroto/e17/blob/master/ecore/src/lib/ecore/ecore_events.c#L272
And the documentation to ecore_event_add also states :
"If @p func_free is NULL, free() will be called with the private structure
pointer."

Let's wait for someone else to comment on the Set/ReleaseCapture changes
you added.

Thanks again,
KaKaRoTo

On Wed, Nov 16, 2011 at 7:43 AM, cnook <kimci...@gmail.com> wrote:

> Dear All,  Hello~
>
> I think you (especially Mr. Vincent, Raster) know this issue.
>
> If user mouse-down on the one of items in elementary_test,
> move(drag) the mouse to the outside of window, and mouse-up,
>
> Then.. it works improperly.. For example..
> You can scroll the list of elementary_test without mouse-down.
>
> The attached patch will resolve this issue.
> Please review the patch and give any feedbacks. Thanks.
>
>
> Sincerely,
> Shinwoo Kim.
>
>
> ------------------------------------------------------------------------------
> RSA(R) Conference 2012
> Save $700 by Nov 18
> Register now
> http://p.sf.net/sfu/rsa-sfdev2dev1
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>
>
------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure 
contains a definitive record of customers, application performance, 
security threats, fraudulent activity, and more. Splunk takes this 
data and makes sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-novd2d
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to