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

Reply via email to