On Sat, Oct 6, 2012 at 2:11 PM, rustyBSD <rusty...@gmx.fr> wrote: > Le 06/10/2012 14:03, Vincent Torri a écrit : >>> Hi, >>> > I looked at the ecore_file_recursive_rm() function >>> > (in ecore/src/lib/ecore_file/), and I wonder why this >>> > is so ugly/complicated. >>> > >>> > We are doing a readlink() and two stat(). Why not simply >>> > do a lstat() ? >> note that lstat does not exist on Windows >> >> Vincent >> > Ah ok, I didn't know. > > But the second stat() is still useless.
Maxime: you can separate code with the _WIN32 macro. Can you update ecore and write a patch that use lstat on non win32 ? Vincent ------------------------------------------------------------------------------ Don't let slow site performance ruin your business. Deploy New Relic APM Deploy New Relic app performance management and know exactly what is happening inside your Ruby, Python, PHP, Java, and .NET app Try New Relic at no cost today and get our sweet Data Nerd shirt too! http://p.sf.net/sfu/newrelic-dev2dev _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel