On Tue, 2011-02-15 at 16:54 +0900, Jaehwan Kim wrote:
> Dear Tom
> 
> > I tried putting the "if" back, and although it fixed this issue, it
> > introduced another one. I will take another look tomorrow.
> Thanks ^^. Please fix this issue.

Sorry for not checking this today, was away in meetings for the better
half of the day, and then had some other things to do, will have to wait
for tomorrow :P

> How about checking the range of wx,wy before it is saved in 
> "_smart_event_mouse_up"?
> I made the patch and tested it. It worked well.
> Please review below patch.

Thanks, it looked good so I applied it and it worked, but there was a
similar issue in other places in the code, and also you forgot to update
wx according to mirroring state. So I modified things a bit and
overhauled those sections, thank you very much though, commit: 57054.

Thanks,
Tom.


------------------------------------------------------------------------------
The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE:
Pinpoint memory and threading errors before they happen.
Find and fix more than 250 security defects in the development cycle.
Locate bottlenecks in serial and parallel code that limit performance.
http://p.sf.net/sfu/intel-dev2devfeb
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to